fix: change how request is "cloned" for loaders (#3218)

* fix: change how request is "cloned" for loaders

* removed derrived properties from init

Author: jacob-ebey

packages/remix-server-runtime/__tests__/handler-test.ts

@@ -0,0 +1,30 @@
+import { json } from "../responses";
+import { createRequestHandler } from "../server";
+
+describe("createRequestHandler", () => {
+  it("retains request headers when stripping body off for loaders", async () => {
+    let handler = createRequestHandler({
+      routes: {
+        root: {
+          id: "routes/test",
+          path: "/test",
+          module: {
+            loader: ({ request }) => json(request.headers.get("X-Foo")),
+          } as any,
+        },
+      },
+      assets: {} as any,
+      entry: { module: {} as any },
+    });
+
+    let response = await handler(
+      new Request("http://.../test", {
+        headers: {
+          "X-Foo": "bar",
+        },
+      })
+    );
+
+    expect(await response.json()).toBe("bar");
+  });
+});

packages/remix-server-runtime/server.ts

@@ -294,8 +294,11 @@ async function handleDocumentRequest({
   }
 
   let loaderRequest = new Request(request.url, {
-    ...request,
     body: null,
+    headers: request.headers,
+    method: request.method,
+    redirect: request.redirect,
+    signal: request.signal,
   });
 
   let routeLoaderResults = await Promise.allSettled(