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

Add the --no-sandbox argument for Chrome Set Headless #3740

Closed
wants to merge 4 commits into from

Conversation

xmtigra
Copy link

@xmtigra xmtigra commented Feb 1, 2024

The last command was added to address an issue related to an error that occurred when using the script. The --no-sandbox argument disables the sandbox mode in Chrome

…at occurred when using the script. The --no-sandbox argument disables the sandbox mode in Chrome
@xmtigra xmtigra requested a review from a team as a code owner February 1, 2024 12:19
Copy link

salesforce-cla bot commented Feb 1, 2024

Thanks for the contribution! Before we can merge this, we need @xmtigra to sign the Salesforce Inc. Contributor License Agreement.

@xmtigra xmtigra closed this Feb 1, 2024
@xmtigra xmtigra reopened this Feb 1, 2024
@xmtigra xmtigra marked this pull request as draft February 2, 2024 13:28
@xmtigra xmtigra marked this pull request as ready for review February 2, 2024 13:29
@jstvz
Copy link
Contributor

jstvz commented May 17, 2024

@xmtigra Thanks for your contribution. We've reviewed the addition of the --no-sandbox flag and decided it's not a change we can merge. Running Chrome without a sandbox is generally not recommended due to security concerns. It's better to run Chrome as a regular user, not as the root user.

If you need to use Chrome without the sandbox in your local setup, you can customize your Robot Framework test suite to add this argument when you open the browser.

I'm closing this PR now. If you have any questions or need to discuss this further, please let us know.

@jstvz jstvz closed this May 17, 2024
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