Fixes https host header default port handling.

mikeal · isaacs · commit 24a1f6ecc502 · 2011-08-07T17:37:56.000-07:00
diff --git a/lib/http2.js b/lib/http2.js
@@ -1259,6 +1259,7 @@ exports.request = function(options, cb) {
   if (options.agent === undefined) {
     options.agent = globalAgent;
   }
+  options.defaultPort = options.defaultPort || 80;
   return new ClientRequest(options, cb);
 };
 
diff --git a/lib/https2.js b/lib/https2.js
@@ -69,6 +69,7 @@ exports.request = function(options, cb) {
   if (options.agent === undefined) {
     options.agent = globalAgent;
   }
+  options.defaultPort = options.defaultPort || 443;
   return http.request(options, cb);
 };
 

Original file line number	Diff line number	Diff line change
`@@ -1259,6 +1259,7 @@ exports.request = function(options, cb) {`
`1259`	`1259`	`if (options.agent === undefined) {`
`1260`	`1260`	`options.agent = globalAgent;`
`1261`	`1261`	`}`
	`1262`	`+ options.defaultPort = options.defaultPort \|\| 80;`
`1262`	`1263`	`return new ClientRequest(options, cb);`
`1263`	`1264`	`};`
`1264`	`1265`
Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,7 @@ exports.request = function(options, cb) {`
`69`	`69`	`if (options.agent === undefined) {`
`70`	`70`	`options.agent = globalAgent;`
`71`	`71`	`}`
	`72`	`+ options.defaultPort = options.defaultPort \|\| 443;`
`72`	`73`	`return http.request(options, cb);`
`73`	`74`	`};`
`74`	`75`