Node ponyfill requests must have an abort signal (#258)

* failing test

* not only

* requests must have an abort signal

* changeset

.changeset/unlucky-lobsters-try.md

@@ -0,0 +1,5 @@
+---
+'@whatwg-node/fetch': patch
+---
+
+Node ponyfill requests must have an abort signal

packages/fetch/dist/create-node-ponyfill.js

@@ -255,6 +255,13 @@ module.exports = function createNodePonyfill(opts = {}) {
             super(requestOrUrl);
           }
           this.formData = getFormDataMethod(formDataModule.File, opts.formDataLimits);
+          this.requestSignal = this.signal;
+          this.optionsSignal = options?.signal || (new ponyfills.AbortController()).signal;
+        }
+        get signal() {
+          // node-fetch does not have a Request.signal
+          // https://github.com/node-fetch/node-fetch/issues/1439
+          return this.requestSignal || this.optionsSignal
         }
       }
       ponyfills.Request = Request;

packages/server/test/request-listener.spec.ts

@@ -213,5 +213,14 @@ describe('Request Listener', () => {
         });
       });
     });
+
+    it('should have the abort signal on the request', async () => {
+      const handler = jest.fn((_request: Request) => new fetchAPI.Response());
+      const adapter = createServerAdapter(handler, fetchAPI.Request);
+
+      await adapter.fetch('http://localhost');
+
+      expect(handler.mock.lastCall?.[0].signal).toBeTruthy();
+    });
   });
 });