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

Use npmrc proxy and strict-ssl settings when downloading deps #185

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mynameisstephen
Copy link

Allows those behind corp firewalls to download required dependencies

@mynameisstephen
Copy link
Author

hi, can we get this review and merged?

@PaulFreund
Copy link

This fixed downloading for me too, only thing I had to change is this -> self and the positions in the file (as the es6 folder is gone) Can we update and merge this please?

@Julusian
Copy link
Collaborator

Julusian commented May 9, 2022

Is this still a problem?
the library used to download files has changed, so it might handle proxies better or might not

@audetto
Copy link

audetto commented Feb 25, 2023

This is still a problem.
The new library is axios which supports proxies, but cannot handle https over an http proxy.
Since most of all traffic is https, this is the same as not working at all.

axios/axios#4766
axios/axios#925

It is unbelievable that in 2023, we are still talking about basic network functionality in node...

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

Successfully merging this pull request may close these issues.

None yet

4 participants