Skip to content

Using curl behind proxy: for nodejs#1371 - #1372

Closed
peter-lyons-kehl wants to merge 3 commits into
nodejs:masterfrom
SeLite:master
Closed

peter-lyons-kehl wants to merge 3 commits into
nodejs:masterfrom
SeLite:master

Conversation

@peter-lyons-kehl

Copy link
Copy Markdown

No description provided.

Comment thread locale/en/download/package-manager.md Outdated

***Note:*** The packages on this page are maintained and supported by their respective packagers, **not** the Node.js core team. Please report any issues you encounter to the package maintainer. If it turns out your issue is a bug in Node.js itself, the maintainer will report the issue upstream.

When running ```curl``` behind a proxy, invoke it as ```curl --proxy http://proxy.racp.edu.au:3128```.

@fhemberger fhemberger Sep 20, 2017 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, not sure if this change is really required, I guess people used to curling behind a proxy usually now their required config switches. Thoughts @nodejs/website?

a) Please use singleticks for inline code.
b) Maybe you should use a more generic proxy URL (something like http://<hostname>:<port> would be okay as well).

@peter-lyons-kehl

Copy link
Copy Markdown
Author

Hi Frederic,

This will save time even for people who have used curl behind a proxy. That's because of 2 stage installation, where in the first step curl downloads a script, which invokes curl again. That second step can be confusing.

a) changed to singleticks for inline code
b) made the instruction proxy-generic

@silverwind

Copy link
Copy Markdown
Contributor

Honestly, I see no need for this. curl honours environment variables like HTTP_PROXY, which is generally the preferred method to enable proxy usage system-wide.

@fhemberger

Copy link
Copy Markdown
Contributor

Personally, I agree with @silverwind, but would like to hear @nodejs/website's thoughts.

@lpinca

lpinca commented Oct 3, 2017

Copy link
Copy Markdown
Member

I also think that the usual way is to use the HTTP_PROXY env variable. I'm -0 on adding this.

@fhemberger fhemberger closed this Oct 5, 2017
@fhemberger

Copy link
Copy Markdown
Contributor

Okay, I've closed this issue. But thanks for your PR, sorry it didn't work out this time.

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.

4 participants