nodejs / node

Node.js JavaScript runtime ✨🐢🚀✨
https://nodejs.org
Other
106.91k stars 29.16k forks source link

Global proxy support #1490

Closed bitinn closed 8 years ago

bitinn commented 9 years ago

Is it possible? Depends on whether we support both http and socks proxy, this can be tricky, because you need to proxy DNS queries as well.

There are times we want io.js http and https to work with a proxy, globally, be it http or socks. With current design, the best solution one can come up with is to patch http and https which often results in compatibility problems (because you need to overwrite default http.request and Agent).

Without a global proxy setting, one cannot easily instruct the whole io.js app to use such a proxy. A notable example is oauth modules, which usually depends on some request modules, and they often use it in a fashion that do not allow proxy settings.

I used these 2 repos to workaround the lack of proxy support, but I feel they are not very sustainable given how fast io.js are improving.

https://github.com/goinstant/global-tunnel https://github.com/yahoo/dnscache

Let me know what you think, it can help a certain group of developers a lot (say ppl in tightly controlled corp network, and ppl in china).

shinnn commented 9 years ago

+1 for proxy support

vvo commented 9 years ago

This would be awesome yes. Wonder how much complex it is.

punmechanic commented 9 years ago

+1, node (and every package I've ever used on node) appears to respect HTTP_PROXY environment variable, iojs does not.

rlidwka commented 9 years ago

I would suggest to make use of https://github.com/nodejs/io.js/pull/881 and run io.js -r write_your_own_proxifier script.js.

I don't think we need neither socks protocol implementation nor http connect tunneling here in core.

TooTallNate commented 8 years ago

Self-plug: https://github.com/TooTallNate/node-proxy-agent

bitinn commented 8 years ago

Other solution I know of (all assume you can pass the http.Agent or are ok with overwriting the global).

https://github.com/floatdrop/proxy-support https://github.com/kevva/caw https://github.com/koichik/node-tunnel

bnoordhuis commented 8 years ago

I don't think proxy support in core is completely out of the question but it's a bit of a slippery slope.

Speaking from prior experience where proxy support was added to an existing product, you start with HTTP CONNECT but you end up supporting everything from SOCKSv5 to HTTP POST, basic/digest/and-so-on auth, Kerberos, NTLMv1 and v2, custom CA chains, client certificates, certificate fingerprinting, etc., etc.

There is no strict need to add it to core either because perfectly good user-land solutions exist.

jasnell commented 8 years ago

@nodejs/ctc ... what do we want to do with this one? The userland options look perfectly good.

bitinn commented 8 years ago

@jasnell Just my 2 cents as the original issue submitter.

ref:

jasnell commented 8 years ago

@bitinn .. understood!

@nodejs/documentation ... perhaps adding some documentation on this would be good?

eljefedelrodeodeljefe commented 8 years ago

Since I ran in this issue the other day: Initially saw a case for core, but after short pondering and discussion w/ @bnoordhuis I regarded @TooTallNate module and others fine for the purpose. Promoting those with docs or moving them under the umbrella might be a better approach, imo, especially with @bnoordhuis's concerns above.

The use case would be also interesting for connecting to Tor, where you don't have many (code) client solution afaik.

Generally I would be a fan of seeing this in core, but it's probably a mistake, dunno.

benjamingr commented 8 years ago

-1 for putting it in core - @bnoordhuis words ring of truth and I share similar experience in scope creep and proxies.

+1 for the docs guiding users for popular userland solutions.

jasnell commented 8 years ago

SGTM. Closing!