fix: Integration test for external redirects (#4627)

* Respect redirects to external url destinations

redirect('https://www.google.com/') should go to http://www.google.com/

We should not prepend the request origin to the destination URL if they
are external.

* Sign CLA

* Revert change to router utils

Co-authored-by: Anthony Powell <anthonypowell@hey.com>

contributors.yml

@@ -65,6 +65,7 @@
 - camiaei
 - CanRau
 - ccssmnn
+- cephalization
 - chaance
 - chaukhoa97
 - chenc041

integration/redirects-test.ts

@@ -144,6 +144,12 @@ test.describe("redirects", () => {
             return <h1>Page 2</h1>
           }
         `,
+        [`app/routes/loader/external.js`]: js`
+          import { redirect } from "@remix-run/node";
+          export const loader = () => {
+            return redirect("https://www.google.com/");
+          }
+        `,
       },
     });
 
@@ -186,4 +192,11 @@ test.describe("redirects", () => {
     // Loader called twice
     await page.waitForSelector(`#count:has-text("3")`);
   });
+
+  test("redirects to external URLs", async ({ page }) => {
+    let app = new PlaywrightFixture(appFixture, page);
+
+    await app.waitForNetworkAfter(() => app.goto("/loader/external"));
+    expect(app.page.url()).toBe("https://www.google.com/");
+  });
 });