url-to-pdf-api icon indicating copy to clipboard operation
url-to-pdf-api copied to clipboard

Updating puppeteer to support newer Chrome features

Open emersonthis opened this issue 2 years ago • 4 comments

Recently I tried to render a page that uses the newish CSS aspect-ratio and the resulting screenshot doesn't recognize it. I suspect the reason is that this project is pinned to puppeteer v2 which predates Chrome support for aspect-ratio.

Puppeteer is currently on v19 and it looks like Chrome added support for aspect-ratio around the time Puppeteer v13 was released. I tried updating package.json to newer versions but the API in those newer versions has changed (in ways I have not studied closely yet) so the app errors out.

emersonthis avatar Oct 27 '22 23:10 emersonthis

@kimmobrunfeldt if I create a PR to update the Puppeteer would you consider merging it?

samuliasmala avatar Mar 20 '23 07:03 samuliasmala

@samuliasmala If for some reason the maintainers don't want to support newer Puppeteer my company would be open to maintaining a fork.

emersonthis avatar Mar 20 '23 17:03 emersonthis

@samuliasmala If for some reason the maintainers don't want to support newer Puppeteer my company would be open to maintaining a fork.

If you do create a maintained fork please let us know 😄

PrimeDominus avatar Apr 12 '23 09:04 PrimeDominus

@samuliasmala @PrimeDominus We just forked the repo: https://github.com/userevidence/url-to-pdf-api

if I create a PR to update the Puppeteer would you consider merging it?

We don't have the bandwidth to tackle this ourselves at the moment, but if anyone has time we'd certainly merge a PR.

emersonthis avatar Apr 17 '23 21:04 emersonthis