url: fix inconsistent port in url.resolveObject

imyller · evanlucas · commit 14c7dcbbcd89 · 2016-08-24T08:37:11.000-05:00
This commit fixes bug where url.resolveObject returns conflicting host and port values. Fixes: #8213 PR-URL: #8214 Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/url.js b/lib/url.js
@@ -776,6 +776,7 @@ Url.prototype.resolveObject = function(relative) {
     // it's absolute.
     if (relative.host || relative.host === '') {
       result.host = relative.host;
+      result.port = relative.port;
       result.auth = null;
     }
     if (relative.hostname || relative.hostname === '') {
diff --git a/test/parallel/test-url.js b/test/parallel/test-url.js
@@ -1540,6 +1540,11 @@ var relativeTests2 = [
    'http://asdf:qwer@www.example.com',
    'http://diff:auth@www.example.com/'],
 
+  // changing port
+  ['https://example.com:81/',
+   'https://example.com:82/',
+   'https://example.com:81/'],
+
   // https://github.com/nodejs/node/issues/1435
   ['https://another.host.com/',
    'https://user:password@example.org/',

Original file line number	Diff line number	Diff line change
`@@ -776,6 +776,7 @@ Url.prototype.resolveObject = function(relative) {`
`776`	`776`	`// it's absolute.`
`777`	`777`	`if (relative.host \|\| relative.host === '') {`
`778`	`778`	`result.host = relative.host;`
	`779`	`+ result.port = relative.port;`
`779`	`780`	`result.auth = null;`
`780`	`781`	`}`
`781`	`782`	`if (relative.hostname \|\| relative.hostname === '') {`