-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
[BUG] NPM 10 ignores proxy configuration via environment variables #6835
Comments
This regression is probably caused by npm/make-fetch-happen#255 |
Seeing the same issue. We're setting http_proxy and https_proxy environment variables, but npm ignores the configuration.
Reverting to node 18 and npm 9.8.1 solves the issue. |
@wraithgar @lukekarrys As you have worked on the |
Thanks for the ping @fgreinacher, I will take a look at this. |
This comment has been minimized.
This comment has been minimized.
For those who are looking for a workaround (not always a solution, depends on your context, only if you really don't rely on environment variables) :
Unlocked us behind a corporate proxy with authentication :) |
This matches the behavior of previous versions of the npm agent. We can't use default parameters since those will only evaluate if 'undefined' is passed in. Fixes npm/cli#6835
This matches the behavior of previous versions of the npm agent. We can't use default parameters since those will only evaluate if 'undefined' is passed in. Fixes npm/cli#6835
Just wanted to mention that node 20 which just became LTS still ships with npm 10.1 and is affected by this issue. This is blocking as we can only use LTS releases. Should the fix be backported or the npm package be updated to 10.2 inside the node distribution? |
Is there an existing issue for this?
This issue exists in the latest npm version
Current Behavior
NPM ignores proxy configuration via environment variables (e.g.
HTTP_PROXY
/HTTPS_PROXY
).Expected Behavior
NPM respects the proxy variables as documented:
Steps To Reproduce
See above.
Environment
The text was updated successfully, but these errors were encountered: