fix: Add support for invalid ipv6 URLs

Closes #114

Author: jhnns

src/from-url.test.ts

@@ -57,6 +57,15 @@ describe(fromUrl.name, () => {
     expect(fromUrl("[1:2:3:4:5:6:7:8]")).toBe("[1:2:3:4:5:6:7:8]");
   });
 
+  // https://github.com/peerigon/parse-domain/issues/114
+  test("it handles URLs with invalid IPv6", () => {
+    expect(fromUrl("http://1:2:3:4:5:6:7:8/path?query")).toBe(
+      "[1:2:3:4:5:6:7:8]"
+    );
+    expect(fromUrl("//1:2:3:4:5:6:7:8")).toBe("[1:2:3:4:5:6:7:8]");
+    expect(fromUrl("1:2:3:4:5:6:7:8")).toBe("[1:2:3:4:5:6:7:8]");
+  });
+
   test("it returns the NO_HOSTNAME symbol for invalid URLs", () => {
     expect(fromUrl(":8080/path?query")).toBe(NO_HOSTNAME);
     expect(fromUrl("/path?query")).toBe(NO_HOSTNAME);

src/from-url.ts

@@ -1,4 +1,5 @@
 const urlPattern = /^[a-z][*+.a-z-]+:\/\//i;
+const invalidIpv6Pattern = /^([a-z][*+.a-z-]+:\/\/)([^[].*:[^/?]*:[^/?]*)(.*)/i;
 
 export const NO_HOSTNAME: unique symbol = Symbol("NO_HOSTNAME");
 
@@ -16,7 +17,7 @@ export const fromUrl = (urlLike: string) => {
   }
 
   // URLs that start with // are protocol relative
-  const url = urlLike.startsWith("//")
+  let url = urlLike.startsWith("//")
     ? `http:${urlLike}`
     : // URLs that start with / do not have a hostname section
     urlLike.startsWith("/")
@@ -25,6 +26,8 @@ export const fromUrl = (urlLike: string) => {
     ? urlLike
     : `http://${urlLike}`;
 
+  url = url.replace(invalidIpv6Pattern, "$1[$2]$3");
+
   try {
     return new URL(url).hostname;
   } catch {