8000 Add WASM_NO_SANDBOX env to disable Chrome sandbox by igolaizola · Pull Request #73 · agnivade/wasmbrowsertest · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add WASM_NO_SANDBOX env to disable Chrome sandbox #73

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

Closed
wants to merge 1 commit into from

Conversation

igolaizola
Copy link

On GitHub Actions (and similar CI environments), Chrome often fails to launch due to its sandbox being unavailable, resulting in the error: "No usable sandbox! If you are running on Ubuntu 23.10+ or another Linux distro that has disabled unprivileged user namespaces..."

This commit adds support for a new environment variable, WASM_NO_SANDBOX, which, when set to "on", appends chromedp.NoSandbox to the Chrome options. This allows Chrome to run without a sandbox and avoids the crash in CI pipelines that lack the necessary permissions or configurations for the standard Chrome sandbox.

On GitHub Actions (and similar CI environments), Chrome often fails to
launch due to its sandbox being unavailable, resulting in the error:
"No usable sandbox! If you are running on Ubuntu 23.10+ or another Linux
distro that has disabled unprivileged user namespaces..."

This commit adds support for a new environment variable,
`WASM_NO_SANDBOX`, which, when set to "on", appends `chromedp.NoSandbox`
to the Chrome options. This allows Chrome to run without a sandbox and
avoids the crash in CI pipelines that lack the necessary permissions or
configurations for the standard Chrome sandbox.
@igolaizola
Copy link
Author

We finally fixed our issues by disabling AppArmor in the CI, same as proposed in this PR: #72

@agnivade
Copy link
Owner

In that case, I guess this PR is no longer needed?

@igolaizola
Copy link
Author

In that case, I guess this PR is no longer needed?

Perhaps there are other use cases where --no-sandbox is needed, but that is no longer the case for us, so I am closing this.

@igolaizola igolaizola closed this Jan 30, 2025
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.

2 participants
0