Redirects when multiple slashes

contributors.yml

@@ -119,6 +119,7 @@
 - eldarshamukhamedov
 - emzoumpo
 - eps1lon
+- ErimTuzcuoglu
 - esamattis
 - evanwinter
 - exegeteio

packages/remix-express/__tests__/server-test.ts

@@ -64,6 +64,18 @@ describe("express createRequestHandler", () => {
       expect(res.headers["x-powered-by"]).toBe("Express");
     });
 
+    it("redirects when multiple slashes", async () => {
+      mockedCreateRequestHandler.mockImplementation(() => async (req) => {
+        return new Response(`${new URL(req.url).pathname}`);
+      });
+
+      let request = supertest(createApp());
+      let res = await request.get("//foo///bar///");
+
+      expect(res.status).toBe(302);
+      expect(res.text).toBe("Found. Redirecting to /foo/bar/");
+    });
+
     it("handles null body", async () => {
       mockedCreateRequestHandler.mockImplementation(() => async () => {
         return new Response(null, { status: 200 });

packages/remix-express/server.ts

@@ -52,6 +52,11 @@ export function createRequestHandler({
     next: express.NextFunction
   ) => {
     try {
+      let cleanUrl = req.url.replace(/\/+/g, '/');
+      if (req.url !== cleanUrl) {
+        return res.redirect(cleanUrl);
+      }
+
       let request = createRemixRequest(req, res);
       let loadContext = getLoadContext?.(req, res);