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

Fixes: #8165 Signed-off-by: Ilkka Myller <ilkka.myller@nodefield.com>
nodejs · Aug 24, 2016 · aaae94e · aaae94e
1 parent 09f861f
commit aaae94e
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
@@ -780,13 +780,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
@@ -1588,6 +1588,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]);