-
Notifications
You must be signed in to change notification settings - Fork 47
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
Pass along proxy environment variables to the browser process #487
Comments
If you're running into an issue getting asset discovery to work with proxy settings, the flag can be passed along to Chromium explicitly via a Percy config file option: version: 2
discovery:
launch-options:
args: ["--proxy-server=XXX"] |
Thank you @wwilsman that work around worked for me. We run Cypress against an IP whitelisted website, where we use a proxy to run the tests against. However, when running Percy the screenshots indicated a broken layout. Running Percy in verbose mode indicated an error Updating the config to pass Chrome the proxy settings resolved the issue. Surprisingly, it also seemed to pass for Firefox (even though Firefox doesn't appear to support passing of a proxy via a command). I think this should probably be added to the documentation too as it has taken a good chunk of time to get to the bottom of this. Thanks again for sharing! |
Hey @AdrianLThomas! We should be fixing in the near future -- this should be handled automatically.
Percy doesn't capture screenshots on your machine. The config that was passed was to the SDKs asset discovery browser, which is there to find & capture the right assets to rerender in Percy's browsers. Here's more info on how Percy works: https://docs.percy.io/docs/debugging-sdks#how-percy-works |
Wonderful, makes sense. Thanks for the response! |
Bumps [actions/stale](https://github.com/actions/stale) from 4 to 5. - [Release notes](https://github.com/actions/stale/releases) - [Changelog](https://github.com/actions/stale/blob/main/CHANGELOG.md) - [Commits](actions/stale@v4...v5) --- updated-dependencies: - dependency-name: actions/stale dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Always thought this was handled automatically by the browser with system-level proxies. But that may not always be the case, or system-level proxies might not always be available for some people.
If proxy env vars are defined, we should pass them along to Chromium explicitly with a flag.
The text was updated successfully, but these errors were encountered: