proxy-from-env icon indicating copy to clipboard operation
proxy-from-env copied to clipboard

Published 20 hours ago • verified publisher icon Rob--W

add support for .npmrc noproxy

Open stefdev49 opened this issue 2 years ago • 5 comments

Add support for 'noproxy' from .npmrc (https://docs.npmjs.com/cli/v9/using-npm/config#noproxy) and have it takes precedence over no_proxy in .npmrc.

noproxy is npm's documented way of specifying non proxied hosts and domains.

stefdev49 avatar Dec 28 '22 09:12 stefdev49

@Rob--W ping

ksathyanm avatar Mar 23 '23 21:03 ksathyanm

@Rob--W axios user here. Looking for noproxy support for npm v9. Can you please take a look at this PR?

ksathyanm avatar Apr 10 '23 00:04 ksathyanm

I have looked into this, and it appears that no_proxy was renamed to noproxy in https://github.com/npm/cli/pull/46 in response to the feedback that the --no-proxy option was not working as expected at https://web.archive.org/web/20211028044200/https://npm.community/t/no-proxy-missing-from-npm-cli/1327.html .

I'll re-examine the addition from #9, update the library if needed + document the npm config variables as appropriate.

Rob--W avatar Apr 15 '23 15:04 Rob--W

As part of maintenance, I'm going to carefully examine the set of changes, and then publish 1.2.0 or even 2.0.0.

The proposed change here has potential to cause breakage, e.g. as seen in #13 after support for npmrc was introduced in #9.

As one of the first steps, I'm going to update the CI in #20.

Due to other obligations I cannot work on this before next month (May).

Rob--W avatar Apr 16 '23 17:04 Rob--W

This PR would help with this issue: https://github.com/cypress-io/cypress/issues/19586

ricksbrown avatar Feb 22 '24 01:02 ricksbrown