-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add support for https proxy #54
Conversation
README.md
Outdated
@@ -159,16 +159,27 @@ or alternatively set variable in `.npmrc` | |||
|
|||
## Download behind proxy | |||
|
|||
In order to be able to download binaries when you're behind a proxy it will be enough to set http_proxy environment variable. | |||
In order to be able to download binaries when you're behind a proxy it will be enough to set the `http_proxy` environment | |||
variable. The `https_proxy` environment variable is supported as well, should the mirror be using the HTTPS protocol. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn't it the proxy server that's using HTTPS, and not the mirror?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess both, right? It would be weird do proxy a HTTP mirror with HTTPS
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
weird but not impossible
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, it's the download helper who will handle the proper proxy to use. it will all depend on the URL you download from. so I think it makes sense the way it's phrased here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a few changes. We can discuss it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Kudos, SonarCloud Quality Gate passed! |
Implemented in #43. Cherry-picked their commit