Tests and correct scheme parsing

Author: eps1lon

packages/next/src/client/resolve-href.ts

@@ -34,7 +34,8 @@ export function resolveHref(
 
   // repeated slashes and backslashes in the URL are considered
   // invalid and will never match a Next.js page/file
-  const urlProtoMatch = urlAsString.match(/^[a-zA-Z+]{1,}:\/\//)
+  // https://datatracker.ietf.org/doc/html/rfc1738#section-2.1
+  const urlProtoMatch = urlAsString.match(/^[a-zA-Z][a-zA-Z+.-]*:\/\//)
   const urlAsStringNoProto = urlProtoMatch
     ? urlAsString.slice(urlProtoMatch[0].length)
     : urlAsString

test/integration/invalid-href/pages/exotic-href.js

@@ -0,0 +1,13 @@
+import Link from 'next/link'
+
+export default function Page() {
+  return (
+    <>
+      <Link href="flatpak+https://dl.flathub.org/repo/appstream/net.krafting.Playlifin.flatpakref">
+        flatpak+https
+      </Link>
+      <Link href="com.apple.tv://">com.apple.tv</Link>
+      <Link href="itms-apps://">itms-apps</Link>
+    </>
+  )
+}

test/integration/invalid-href/pages/second.js

@@ -12,16 +12,16 @@ export default function Page() {
       id="click-me"
       onClick={(e) => {
         e.preventDefault()
+        // this should throw an error on load since prefetch
+        // receives the invalid href
         router[method](invalidLink)
       }}
     >
       invalid link :o
     </a>
   ) : (
-    // this should throw an error on load since prefetch
-    // receives the invalid href
     <Link href={invalidLink} id="click-me">
-      invalid link :o
+      valid link :o
     </Link>
   )
 }

test/integration/invalid-href/test/index.test.js

@@ -98,10 +98,14 @@ describe('Invalid hrefs', () => {
         await noError('/first')
       })
 
-      it('does not show error in production when https://google.com is used as href on Link', async () => {
+      it('does not show error in production when https:// is used in href on Link', async () => {
         await noError('/second')
       })
 
+      it('does not show error in production when exotic protocols are used in href in Link', async () => {
+        await noError('/exotic-href')
+      })
+
       it('does not show error when internal href is used with external as', async () => {
         await noError('/invalid-relative', true)
       })
@@ -160,10 +164,14 @@ describe('Invalid hrefs', () => {
         await noError('/first')
       })
 
-      it('does not show error when https://google.com is used as href on Link', async () => {
+      it('does not show error when https:// is used as href in Link', async () => {
         await noError('/second')
       })
 
+      it('does not show error when exotic protocols are used in href in Link', async () => {
+        await noError('/exotic-href')
+      })
+
       // eslint-disable-next-line jest/no-identical-title
       it('shows error when dynamic route mismatch is used on Link', async () => {
         await showsError(