Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PDF rendering has broken recently #73

Closed
spkane opened this issue Feb 16, 2019 · 4 comments
Closed

PDF rendering has broken recently #73

spkane opened this issue Feb 16, 2019 · 4 comments

Comments

@spkane
Copy link

spkane commented Feb 16, 2019

I did a little work to narrow this down, but sometime after this commit 3fe441cf7a406416300735b0f5de4a6e48a6ff69 the PDF rendering started hanging (at least in the docker container). Building the Docker container from this commit, still works.

On HEAD, adding --pdf to the command line causes the container to just hang while rendering.

docker run --rm -v $PWD:/home/marp/app/ marpteam/marp-cli:latest target.md --theme-set spkane.css --pdf --allow-local-files
[  INFO ] Converting 1 markdown...
[  WARN ] Insecure local file accessing is enabled for conversion of target.pdf.

Building from an commit about a month or two older results in a working PDF render:

docker run --rm -v $PWD:/home/marp/app/ 46c6ff46a3d9 target.md --theme-set spkane.css --pdf --allow-local-files
[  INFO ] Converting 1 markdown...
[  WARN ] Insecure local file accessing is enabled for conversion of target.pdf.
[  INFO ] target.md => target.pdf
@yhatt
Copy link
Member

yhatt commented Feb 16, 2019

Hmm, I guess mismatch between supported Chromium version in Marp CLI's Puppeteer and installed Chromium version in Docker image.

$ docker run --rm --entrypoint="" marpteam/marp-cli chromium-browser --version
Chromium 71.0.3578.98 

Puppeteer's troubleshooting says:

The newest Chromium package supported on Alpine is 71, which was corresponding to Puppeteer v1.9.0.

@yhatt
Copy link
Member

yhatt commented Feb 16, 2019

I submitted #74 to fix to use matched Puppeteer version in Docker image. I've confirmed working PDF conversion in built image correctly.

I'll merge it and update latest image if it's looks like OK.

@yhatt yhatt closed this as completed in #74 Feb 17, 2019
@yhatt
Copy link
Member

yhatt commented Feb 17, 2019

Updated Docker image to work PDF conversion. Try running docker pull marpteam/marp-cli and converting your PDF again.

@spkane
Copy link
Author

spkane commented Feb 17, 2019

That appears to have fixed it. Thank you for the quick turnaround!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants