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: Disable sandboxing for cypress_test rules #66

Merged
merged 2 commits into from
May 1, 2024

Conversation

mrmeku
Copy link
Contributor

@mrmeku mrmeku commented Apr 16, 2024

Cypress has moved away from supporting the env
variable RUN_ELECTRON_AS_NODE which makes it
hard to keep up with their electron server's
sandboxing violations. We now disable the sandbox
so that user's can don't run into these issues.


Type of change

  • Bug fix (change which fixes an issue)

Test plan

  • New test cases added
    (using a cypress version 13.6.6 required this change on macOS)

@mrmeku mrmeku requested a review from gregmagolan April 16, 2024 04:49
@mrmeku mrmeku force-pushed the no-sandbox branch 6 times, most recently from b6b6d13 to 11b4b2e Compare April 16, 2024 05:44
@mrmeku mrmeku marked this pull request as ready for review April 16, 2024 15:42
cypress/defs.bzl Outdated Show resolved Hide resolved
e2e/workspace/MODULE.bazel Outdated Show resolved Hide resolved
@mrmeku mrmeku requested review from jbedard and removed request for thesayyn April 22, 2024 19:40
cypress/defs.bzl Outdated Show resolved Hide resolved
@mrmeku mrmeku force-pushed the no-sandbox branch 3 times, most recently from f85544d to eda9d77 Compare April 29, 2024 01:53
MODULE.bazel Outdated Show resolved Hide resolved
cypress/defs.bzl Outdated Show resolved Hide resolved
Copy link
Contributor Author

mrmeku commented Apr 30, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mrmeku and the rest of your teammates on Graphite Graphite

variable RUN_ELECTRON_AS_NODE which makes it
hard to keep up with their electron server's
sandboxing violations. We now disable the sandbox
so that user's can don't run into these issues.
- Switch to `disable_sandbox` defaulting to true
- Removed unused attribute from rule impl
@gregmagolan gregmagolan merged commit 6e9bd95 into aspect-build:main May 1, 2024
8 checks passed
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.

4 participants