fix(server): fix url join (#166)

Author: gregberge

packages/server/src/util.js

@@ -9,5 +9,7 @@ export const smartRequire = modulePath => {
   return require(modulePath)
 }
 
-export const joinURLPath = (...paths) =>
-  paths.join('/').replace(/(?<!http(s)?:)(?<=.)\/\//g, '/')
+export const joinURLPath = (...paths) => {
+  const cleanPaths = paths.map(path => path.replace(/\/$/, ''))
+  return cleanPaths.join('/')
+}

packages/server/src/util.test.js

@@ -6,20 +6,27 @@ describe('util', () => {
       expect(joinURLPath('public', 'style.css')).toBe('public/style.css')
       expect(joinURLPath('public/', 'style.css')).toBe('public/style.css')
     })
+
+    it('should join paths starting with "/"', () => {
+      expect(joinURLPath('/foo', 'style.css')).toBe('/foo/style.css')
+      expect(joinURLPath('/', 'style.css')).toBe('/style.css')
+    })
+
     it('should join paths with absolute public path', () => {
       const publicPath = 'http://localhost:3001/public'
 
       expect(joinURLPath(publicPath, 'style.css')).toBe(
-        `${publicPath}/style.css`,
+        `http://localhost:3001/public/style.css`,
       )
       expect(joinURLPath(`${publicPath}/`, 'style.css')).toBe(
-        `${publicPath}/style.css`,
+        `http://localhost:3001/public/style.css`,
       )
     })
+
     it('should join paths with protocol free public path', () => {
       const publicPath = '//127.0.0.1/public'
       expect(joinURLPath(publicPath, 'style.css')).toBe(
-        `${publicPath}/style.css`,
+        `//127.0.0.1/public/style.css`,
       )
     })
   })