fix request signal not aborting after request is garbage collected

packages/remix-server-runtime/data.ts

@@ -42,6 +42,8 @@ export async function callRouteAction({
   routeId: string;
   singleFetch: boolean;
 }) {
+  makeRequestSignalGcSafe(request);
+
   let result = await action({
     request: singleFetch
       ? stripRoutesParam(stripIndexParam(request))
@@ -80,6 +82,8 @@ export async function callRouteLoader({
   routeId: string;
   singleFetch: boolean;
 }) {
+  makeRequestSignalGcSafe(request);
+
   let result = await loader({
     request: singleFetch
       ? stripRoutesParam(stripIndexParam(request))
@@ -113,6 +117,15 @@ export async function callRouteLoader({
   return isResponse(result) ? result : json(result);
 }
 
+function makeRequestSignalGcSafe(request: Request) {
+  // `undici` wraps the signal passed to the `Request` constructor. When the
+  // request object is garbage collected, the signal stops working. This is
+  // problematic when a loader or action is waiting for a signal to be aborted.
+  // To fix this, we hold a reference to the request in the signal.
+  Object.defineProperty(request.signal, "__request", request);
+  return request.signal;
+}
+
 // TODO: Document these search params better
 // and stop stripping these in V2. These break
 // support for running in a SW and also expose
@@ -136,7 +149,7 @@ function stripIndexParam(request: Request) {
     method: request.method,
     body: request.body,
     headers: request.headers,
-    signal: request.signal,
+    signal: makeRequestSignalGcSafe(request),
   };
 
   if (init.body) {
@@ -153,7 +166,7 @@ function stripDataParam(request: Request) {
     method: request.method,
     body: request.body,
     headers: request.headers,
-    signal: request.signal,
+    signal: makeRequestSignalGcSafe(request),
   };
 
   if (init.body) {
@@ -170,7 +183,7 @@ function stripRoutesParam(request: Request) {
     method: request.method,
     body: request.body,
     headers: request.headers,
-    signal: request.signal,
+    signal: makeRequestSignalGcSafe(request),
   };
 
   if (init.body) {