Skip to content

Commit e093444

Browse files
committed
https-proxy-agent: comments
1 parent 627ffb3 commit e093444

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

https-proxy-agent.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,12 @@ function HttpsProxyAgent (opts) {
3131

3232
this.secure = proxy.protocol && proxy.protocol == 'https:';
3333

34+
// prefer `hostname` over `host`, and set the `port` if needed
3435
proxy.host = proxy.hostname || proxy.host;
3536
proxy.port = +proxy.port || (this.secure ? 443 : 80);
3637

3738
if (proxy.host && proxy.path) {
38-
// XXX: if both a `host` and `path` are specified then it's most likely the
39+
// if both a `host` and `path` are specified then it's most likely the
3940
// result of a `url.parse()` call... we need to remove the `path` portion so
4041
// that `net.connect()` doesn't attempt to open that as a unix socket file.
4142
delete proxy.path;

0 commit comments

Comments
 (0)