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

Retry spawning Chromium with WebSocket connection if failed to connect through pipe #400

Merged
merged 3 commits into from
Nov 6, 2021

Conversation

yhatt
Copy link
Member

@yhatt yhatt commented Nov 6, 2021

If failed to connect to Chromium with pipe option with a known error (Target.setDiscoverTargets), Marp CLI will retry to spawn chromium with using WebSockets instead (pipe: false).

It might mitigate some conversion issues reported in marp-team/marp#165 and marp-team/marp#198. Close #395.

This PR is also including refactor for #317. The warning had no longer outputted in the snap binary that was installed by apt due to the shebang script for proxy.

If failed to connect to Chromium with pipe option with a known error,
Marp CLI will retry to spawn chromium with using WebSockets instead.
@yhatt yhatt merged commit 31ba07d into main Nov 6, 2021
@yhatt yhatt deleted the pipe-fallback branch November 6, 2021 18:30
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

Successfully merging this pull request may close these issues.

Improve reliability of connection to Chrome process
1 participant