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

[Task]: Change Chrome Docker image #571

Closed
wants to merge 2 commits into from
Closed

[Task]: Change Chrome Docker image #571

wants to merge 2 commits into from

Conversation

kingjia90
Copy link
Contributor

@kingjia90 kingjia90 commented Mar 11, 2024

It seems that it got some breaking changes, by changing to chromium it has some BC layer fallback, see
https://github.com/browserless/browserless/blob/8777c638001c69b44928e2906f86a28ed9d79313/CHANGELOG.md?plain=1#L33

@kingjia90
Copy link
Contributor Author

kingjia90 commented Mar 11, 2024

Potentially we have to stick with v1 , as keep-alive is not supported anymore
https://github.com/browserless/browserless/blob/8777c638001c69b44928e2906f86a28ed9d79313/CHANGELOG.md?plain=1#L75 but not sure if that's the keep alive that i am thinking of

However with v1, had other issues as chrome-php/chrome#611 (comment)

@kingjia90 kingjia90 closed this Mar 29, 2024
@kingjia90 kingjia90 deleted the chromium-image branch March 29, 2024 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants