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

Fix passing of --no-sandbox to chrome-option #51

Merged
merged 1 commit into from
Jun 19, 2019

Conversation

gdancel
Copy link
Contributor

@gdancel gdancel commented May 22, 2019

No description provided.

@gutschilla
Copy link
Owner

@gdancel - I stumbled upon the same thing. It's an issue with Elixir's parameter passing which is fixed in Elixir 1.8 - at least for me.

Which Elixir version are you using. For Elixir 1.7 the equal sign is required. Unfortunately, for 1.8 it will break the command.

Maybe I should introduce a version switch for a while after the version-dependency has been verified.

What do you think? Please let me know of your environment.

@gdancel
Copy link
Contributor Author

gdancel commented Jun 4, 2019

@gutschilla I'm using 1.6.6. I didn't know that it will break in 1.8. The workaround I ended up using is adding shell_params: ["--chrome-option=--no-sandbox"] in the options for generate. Since there's a way to disable sandbox without using app env I think it's up to you if you'd like to support old versions being apple to use app env to disable sandbox.

@gutschilla
Copy link
Owner

@gdancel Thanks for coming back to me on this issue. Since there's a workaround in place I'd say I will document this. I would rather not try to bump the minimum Elixir version for this. I basic docker scenarios, the no-sandbox option is pretty handy as you need to run as root from within the container - so at least for me it's important.

@gutschilla
Copy link
Owner

I am adding a "known issue" section right now. I will bump the minimum Elixir version to 1.7 for 0.6.0

Copy link
Owner

@gutschilla gutschilla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Whatever I have seen on my boxes, with OTP 22 it works like that, again. So I am going to include this. Thanks for contributing!

@gutschilla gutschilla merged commit 7c09a26 into gutschilla:master Jun 19, 2019
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