-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 #1978 - ability to deploy to networks behind a corporate proxies #3280
Fix #1978 - ability to deploy to networks behind a corporate proxies #3280
Conversation
…porate proxies
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
This issue is also being tracked on Linear. We use Linear to manage our development process, but we keep the conversations on Github. LINEAR-ID: f06ca137-d6a5-4478-8920-9e47e5e8f233 |
Pinging @fvictorio |
same error!! |
What is the error ? |
|
GlobalDispatcher does not work on specific proxies. This PR looks at HTTP_PROXY/HTTPS_PROXY and it sets ProxyAgent for connection. undici connection pools does not support proxy yet. |
Any update on this PR @fvictorio ? |
Hey @umakanth-fmr, I haven't had time to review this yet, sorry 😞 I skimmed the PR's code and it looks good, but since this is a very sensitive change, I'll need to make a careful review before merging it. |
Hey @umakanth-fmr, I ended up going for a somewhat different approach in #3432. The main difference is that I'm using a single environment variable, I'm going to close this PR in favor of that one. Thanks again and sorry for not merging yours. |
@gitpoap-bot @umakanth-fmr deserves a gitpoap |
Congrats, @umakanth-fmr ! You've earned a GitPOAP for your contribution! GitPOAP: 2022 Hardhat Contributor: Head to gitpoap.io & connect your GitHub account to mint! Learn more about GitPOAPs here. |
This PR addresses the issue with connecting to 3rd party networks behind corporate proxy. Currently undici Pool's does not support proxy. (nodejs/undici#1270).
We patched our local harhat to make it work.
Relevant issues on hardhat issue tracker.
#1978