[fix] Add a leading slash only if the URL is special

If the value of the `pathname` property does not start with a `/`, add it only if the URL is special.
unshiftio · Jul 24, 2021 · fed6d9e · fed6d9e
1 parent 94872e7
commit fed6d9e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/index.js b/index.js
@@ -347,10 +347,7 @@ function Url(address, location, parser) {
   // Default to a / for pathname if none exists. This normalizes the URL
   // to always have a /
   //
-  if (
-      url.pathname.charAt(0) !== '/'
-    && (url.hostname || url.protocol === 'file:')
-  ) {
+  if (url.pathname.charAt(0) !== '/' && isSpecial(url.protocol)) {
     url.pathname = '/' + url.pathname;
   }
 

diff --git a/test/test.js b/test/test.js
@@ -361,8 +361,8 @@ describe('url-parse', function () {
     url = 'foo://example.com';
     parsed = parse(url);
     assume(parsed.hostname).equals('example.com');
-    assume(parsed.pathname).equals('/');
-    assume(parsed.href).equals('foo://example.com/');
+    assume(parsed.pathname).equals('');
+    assume(parsed.href).equals('foo://example.com');
     assume(parsed.slashes).is.true();
 
     url = 'foo:///example.com';