fix(server): exclude http and https from regex (#155)

* Exclude http and https from regex. Should fix #153

* Fix regex whitespace and simplify

* Add test for joinURLPath

packages/server/src/util.js

@@ -9,4 +9,5 @@ export const smartRequire = modulePath => {
   return require(modulePath)
 }
 
-export const joinURLPath = (...paths) => paths.join('/').replace(/\/\//g, '/')
+export const joinURLPath = (...paths) =>
+  paths.join('/').replace(/(?<!:)\/\//g, '/')

packages/server/src/util.test.js

@@ -0,0 +1,20 @@
+import { joinURLPath } from './util'
+
+describe('util', () => {
+  describe('#joinURLPath', () => {
+    it('should join paths with relative public path', () => {
+      expect(joinURLPath('public', 'style.css')).toBe('public/style.css')
+      expect(joinURLPath('public/', 'style.css')).toBe('public/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`,
+      )
+      expect(joinURLPath(`${publicPath}/`, 'style.css')).toBe(
+        `${publicPath}/style.css`,
+      )
+    })
+  })
+})