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

Proxy Support Feature #62

Closed
randomactsofsecurity opened this issue Sep 18, 2020 · 4 comments
Closed

Proxy Support Feature #62

randomactsofsecurity opened this issue Sep 18, 2020 · 4 comments

Comments

@randomactsofsecurity
Copy link
Contributor

Describe the solution you'd like
Add HTTP and SOCKS5 proxy support screenshot functionality

Describe alternatives you've considered
Related to the old issue #21

Additional context
Seems like chromedp has this built in support already, just need to stub in the options and test in chrome/chrome.go

https://github.com/chromedp/examples/pull/20/files/0bfcebe721cd401786649f72d0b7926369348623#diff-a5b9dfd3fa592c284ba488a7c748679e

@leonjza
Copy link
Member

leonjza commented Sep 18, 2020

I played with this a bit last night actually and was having trouble getting a proxy configuration to work for Chrome. Maybe someone else could test it as well?

@randomactsofsecurity
Copy link
Contributor Author

the chromedp proxyserver works fine, it's actually the preflight http.client that will need proxy support added as well.

I've got a local copy working with socks5, just need to allow it to work with http. Will make a PR soon

@randomactsofsecurity
Copy link
Contributor Author

randomactsofsecurity commented Sep 20, 2020

Added proxy support at PR #65

I've tested without proxy, with a socks 5 and with a HTTP proxy and it seems to work for me. Should be good with a bit more validation on your part. I've also made it so it follows the global timeout argument, in case someone needs to delay the connection more.

Also I haven't tested, but it may also support environment variables as well even if the proxy-server isn't set, at least for the preflight checks

@randomactsofsecurity
Copy link
Contributor Author

closed, merged in 9736345

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants