fix: correctProtocol misparsing protocol

Author: markovejnovic

lib/parse-url.js

@@ -21,6 +21,11 @@ const correctProtocol = (arg, protocols) => {
     return arg
   }
 
+  const doubleSlash = arg.indexOf('//')
+  if (doubleSlash === firstColon + 1) {
+    return arg
+  }
+
   const firstAt = arg.indexOf('@')
   if (firstAt > -1) {
     if (firstAt > firstColon) {
@@ -30,11 +35,6 @@ const correctProtocol = (arg, protocols) => {
     }
   }
 
-  const doubleSlash = arg.indexOf('//')
-  if (doubleSlash === firstColon + 1) {
-    return arg
-  }
-
   return `${arg.slice(0, firstColon + 1)}//${arg.slice(firstColon + 1)}`
 }
 

test/parse-url.js

@@ -15,3 +15,10 @@ t.test('can parse file urls', async t => {
   t.ok(parseUrl(u))
   t.ok(HostedGit.parseUrl(u))
 })
+
+t.test('can parse custom urls', async t => {
+  const u = 'foobar://baz.com:user/repo'
+  t.ok(parseUrl(u))
+  t.equal(parseUrl(u).protocol, 'foobar:')
+  t.ok(HostedGit.parseUrl(u))
+})