url: keep auth in url.resolve() if host matches

Fixes: #8165 PR-URL: #8215 Reviewed-By: James M Snell <jasnell@gmail.com>
nodejs · Sep 9, 2016 · 6ae7af0 · 6ae7af0
1 parent eec5d02
commit 6ae7af0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lib/url.js b/lib/url.js
@@ -775,13 +775,13 @@ Url.prototype.resolveObject = function(relative) {
   if (isRelAbs) {
     // it's absolute.
     if (relative.host || relative.host === '') {
+      if (result.host !== relative.host) result.auth = null;
       result.host = relative.host;
       result.port = relative.port;
-      result.auth = null;
     }
     if (relative.hostname || relative.hostname === '') {
+      if (result.hostname !== relative.hostname) result.auth = null;
       result.hostname = relative.hostname;
-      result.auth = null;
     }
     result.search = relative.search;
     result.query = relative.query;

diff --git a/test/parallel/test-url.js b/test/parallel/test-url.js
@@ -1571,6 +1571,9 @@ var relativeTests2 = [
   ['mailto:another.host.com',
    'mailto:user@example.org',
    'mailto:another.host.com'],
+  ['https://example.com/foo',
+   'https://user:password@example.com',
+   'https://user:password@example.com/foo'],
 ];
 relativeTests2.forEach(function(relativeTest) {
   const a = url.resolve(relativeTest[1], relativeTest[0]);