From 4f817cad4ba7d5c8e2f7c08125b80598c067af47 Mon Sep 17 00:00:00 2001
From: Luke Walker <49527534+lkwr@users.noreply.github.com>
Date: Fri, 27 Jan 2023 19:34:12 +0100
Subject: [PATCH 1/2] Added better detection for absolute urls in link
 component

---
 .../__tests__/link-href-test.tsx              | 44 +++++++++++++++++++
 packages/react-router-dom/index.tsx           |  3 +-
 2 files changed, 45 insertions(+), 2 deletions(-)

diff --git a/packages/react-router-dom/__tests__/link-href-test.tsx b/packages/react-router-dom/__tests__/link-href-test.tsx
index 92b2e828e9..a37d7a7b7d 100644
--- a/packages/react-router-dom/__tests__/link-href-test.tsx
+++ b/packages/react-router-dom/__tests__/link-href-test.tsx
@@ -128,6 +128,50 @@ describe("<Link> href", () => {
 
       expect(renderer.root.findByType("a").props.href).toEqual("//remix.run");
     });
+
+    test('<Link to="mailto:remix@example.com"> is treated as external link', () => {
+      let renderer: TestRenderer.ReactTestRenderer;
+      TestRenderer.act(() => {
+        renderer = TestRenderer.create(
+          <MemoryRouter initialEntries={["/inbox/messages"]}>
+            <Routes>
+              <Route path="inbox">
+                <Route
+                  path="messages"
+                  element={<Link to="mailto:remix@example.com" />}
+                />
+              </Route>
+            </Routes>
+          </MemoryRouter>
+        );
+      });
+
+      expect(renderer.root.findByType("a").props.href).toEqual(
+        "mailto:remix@example.com"
+      );
+    });
+
+    test('<Link to="web+remix://somepath"> is treated as external link', () => {
+      let renderer: TestRenderer.ReactTestRenderer;
+      TestRenderer.act(() => {
+        renderer = TestRenderer.create(
+          <MemoryRouter initialEntries={["/inbox/messages"]}>
+            <Routes>
+              <Route path="inbox">
+                <Route
+                  path="messages"
+                  element={<Link to="web+remix://somepath" />}
+                />
+              </Route>
+            </Routes>
+          </MemoryRouter>
+        );
+      });
+
+      expect(renderer.root.findByType("a").props.href).toEqual(
+        "web+remix://somepath"
+      );
+    });
   });
 
   describe("in a dynamic route", () => {
diff --git a/packages/react-router-dom/index.tsx b/packages/react-router-dom/index.tsx
index 73748ff5c2..a2adb57978 100644
--- a/packages/react-router-dom/index.tsx
+++ b/packages/react-router-dom/index.tsx
@@ -420,8 +420,7 @@ export const Link = React.forwardRef<HTMLAnchorElement, LinkProps>(
   ) {
     // `location` is the unaltered href we will render in the <a> tag for absolute URLs
     let location = typeof to === "string" ? to : createPath(to);
-    let isAbsolute =
-      /^[a-z+]+:\/\//i.test(location) || location.startsWith("//");
+    let isAbsolute = /^(?:[a-z][a-z0-9+.-]*:|\/\/)/i.test(location);
 
     // Location to use in the click handler
     let navigationLocation = location;

From 9400e9a37d7a3f2fa461cde21ea4c70d6429e080 Mon Sep 17 00:00:00 2001
From: Luke Walker <49527534+lkwr@users.noreply.github.com>
Date: Mon, 30 Jan 2023 22:54:14 +0100
Subject: [PATCH 2/2] added changeset

---
 .changeset/afraid-scissors-rhyme.md | 5 +++++
 1 file changed, 5 insertions(+)
 create mode 100644 .changeset/afraid-scissors-rhyme.md

diff --git a/.changeset/afraid-scissors-rhyme.md b/.changeset/afraid-scissors-rhyme.md
new file mode 100644
index 0000000000..e9df5c9ad3
--- /dev/null
+++ b/.changeset/afraid-scissors-rhyme.md
@@ -0,0 +1,5 @@
+---
+"react-router-dom": patch
+---
+
+Improved absolute url detection in `Link` component (now also supports `mailto:` urls)