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 ability to run in sandbox mode #18

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

jcblw
Copy link
Member

@jcblw jcblw commented Oct 27, 2019

Description

Before this, there was an assumption that all usage of this action would be ran un the --no-sandbox flag. This is not recommended by the Puppeteer team. This implements creating a user and allowing that user access to all the resources needed to run the test.

closes #17

@jcblw
Copy link
Member Author

jcblw commented Oct 27, 2019

Running into issues getting the new sandboxed user access to the workflow directory.

See mujo-code/source/runs/276297037

@jcblw
Copy link
Member Author

jcblw commented Oct 27, 2019

@stefanbuck or @bantic if y'all have any ideas to get around this I'm all ears 😁

@stefanbuck
Copy link
Collaborator

I'm sorry for the very late reply. Somehow this message slipped through my inbox. I don't see any issue with adding such a user.

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.

Cannot run Puppeteer in sandbox mode?
2 participants