fix(joinURL): don't allow double '//' when single slashes are passed (#…

…13)
unjs · Feb 8, 2021 · e44fe2b · e44fe2b
1 parent bb2a64a
commit e44fe2b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/utils.ts b/src/utils.ts
@@ -42,7 +42,10 @@ export function joinURL (base: string, ...input: string[]): string {
   let url = base || ''
 
   for (const i of input) {
-    url = withTrailingSlash(url) + withoutLeadingSlash(i)
+    const part = withoutLeadingSlash(i)
+    if (part !== '/') {
+      url = withTrailingSlash(url) + part
+    }
   }
 
   return url

diff --git a/test/join.test.ts b/test/join.test.ts
@@ -7,6 +7,7 @@ describe('joinURL', () => {
     { input: ['/'], out: '/' },
     { input: ['/a'], out: '/a' },
     { input: ['a', 'b'], out: 'a/b' },
+    { input: ['/', '/b'], out: '/b' },
     { input: ['a', 'b/', 'c'], out: 'a/b/c' },
     { input: ['a', 'b/', '/c'], out: 'a/b/c' }
   ]