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

adding support for no_proxy environment variable #69

Closed
jacklty opened this issue Apr 15, 2019 · 3 comments · Fixed by #361
Closed

adding support for no_proxy environment variable #69

jacklty opened this issue Apr 15, 2019 · 3 comments · Fixed by #361
Assignees
Labels
type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.

Comments

@jacklty
Copy link

jacklty commented Apr 15, 2019

Could you add support for no_proxy environment variable please? so we are able to bypass proxy for specific url pattern?

Thanks

FYI: Similar request was made to axios as well but it yet not be part of official release - axios/axios#434

@jacklty jacklty changed the title adding support for no_proxy environment adding support for no_proxy environment variable Apr 15, 2019
@yoshi-automation yoshi-automation added 🚨 This issue needs some love. triage me labels May 1, 2019
@JustinBeckwith JustinBeckwith added type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. and removed 🚨 This issue needs some love. enhancement labels May 13, 2019
@csvan
Copy link

csvan commented Oct 31, 2020

This completely breaks gaxios on machines behind corporate proxies. Is there no way to work around this?

@bcoe
Copy link
Contributor

bcoe commented Nov 2, 2020

@csvan @jacklty could you describe the expected behavior of no_proxy with a bit more detail?

I gather you provide a list of domains that should not have https_proxy or http_proxy populated for them?

@csvan
Copy link

csvan commented Nov 2, 2020

@bcoe precisely, it could look like this:

export no_proxy=localhost,127.0.0.0/8,::1,.mysub.mydomain.com,myotherdomain.net

Among others, node-fetch (at least seems) to respect it when used on its own.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants