@whatwg-node/node-fetch 0.7.23-alpha-20250726023513-1ae33d44a98b7e7238a179b7b100c6bdb1de96e7 → 0.7.23-alpha-20250726084802-683ac7e676415a5d08903fcace71146b2a74a837

package/cjs/Body.js

@@ -27,11 +27,9 @@ class PonyfillBody {
     bodyUsed = false;
     contentType = null;
     contentLength = null;
-    _signal = null;
     constructor(bodyInit, options = {}) {
         this.bodyInit = bodyInit;
         this.options = options;
-        this._signal = options.signal || null;
         const { bodyFactory, contentType, contentLength, bodyType, buffer } = processBodyInit(bodyInit);
         this._bodyFactory = bodyFactory;
         this.contentType = contentType;
@@ -221,9 +219,6 @@ class PonyfillBody {
                 limits: formDataLimits,
                 defCharset: 'utf-8',
             });
-            if (this._signal) {
-                (0, node_stream_1.addAbortSignal)(this._signal, bb);
-            }
             let completed = false;
             const complete = (err) => {
                 if (completed)

package/cjs/Request.js

@@ -76,6 +76,7 @@ class PonyfillRequest extends Body_js_1.PonyfillBody {
                 this.agent = requestInit.agent;
             }
         }
+        this._signal = requestInit?.signal || undefined;
     }
     headersSerializer;
     cache;
@@ -91,6 +92,7 @@ class PonyfillRequest extends Body_js_1.PonyfillBody {
     referrer;
     referrerPolicy;
     _url;
+    _signal;
     get signal() {
         this._signal ||= new AbortController().signal;
         return this._signal;

package/cjs/fetchCurl.js

@@ -22,7 +22,7 @@ function fetchCurl(fetchRequest) {
     }
     curlHandle.enable(CurlFeature.StreamResponse);
     let signal;
-    if (fetchRequest._signal == null) {
+    if (fetchRequest._signal === null) {
         signal = undefined;
     }
     else if (fetchRequest._signal) {
@@ -104,12 +104,8 @@ function fetchCurl(fetchRequest) {
         }
     });
     curlHandle.once('stream', function streamListener(stream, status, headersBuf) {
-        const passThrough = new node_stream_1.PassThrough();
-        (0, utils_js_1.pipeThrough)({
-            src: stream,
-            dest: passThrough,
-            signal,
-            onError: deferredPromise.reject,
+        const outputStream = stream.pipe(new node_stream_1.PassThrough(), {
+            end: true,
         });
         const headersFlat = headersBuf
             .toString('utf8')
@@ -122,7 +118,7 @@ function fetchCurl(fetchRequest) {
                     if (!stream.destroyed) {
                         stream.resume();
                     }
-                    passThrough.destroy();
+                    outputStream.destroy();
                     deferredPromise.reject(new Error('redirect is not allowed'));
                 }
                 return true;
@@ -130,14 +126,14 @@ function fetchCurl(fetchRequest) {
             return false;
         });
         const headersInit = headersFlat.map(headerFlat => headerFlat.split(/:\s(.+)/).slice(0, 2));
-        const ponyfillResponse = new Response_js_1.PonyfillResponse(passThrough, {
+        const ponyfillResponse = new Response_js_1.PonyfillResponse(outputStream, {
             status,
             headers: headersInit,
             url: curlHandle.getInfo(Curl.info.REDIRECT_URL)?.toString() || fetchRequest.url,
             redirected: Number(curlHandle.getInfo(Curl.info.REDIRECT_COUNT)) > 0,
         });
         deferredPromise.resolve(ponyfillResponse);
-        streamResolved = passThrough;
+        streamResolved = outputStream;
     });
     setImmediate(() => {
         curlHandle.perform();

package/cjs/fetchNodeHttp.js

@@ -92,24 +92,20 @@ function fetchNodeHttp(fetchRequest) {
                         return;
                     }
                 }
-                outputStream ||= new node_stream_1.PassThrough();
-                (0, utils_js_1.pipeThrough)({
-                    src: nodeResponse,
-                    dest: outputStream,
+                outputStream ||= new node_stream_1.PassThrough({
                     signal,
-                    onError: reject,
                 });
+                nodeResponse.pipe(outputStream);
                 const statusCode = nodeResponse.statusCode || 200;
                 let statusText = nodeResponse.statusMessage || node_http_1.STATUS_CODES[statusCode];
                 if (statusText == null) {
                     statusText = '';
                 }
-                const ponyfillResponse = new Response_js_1.PonyfillResponse(outputStream, {
+                const ponyfillResponse = new Response_js_1.PonyfillResponse(outputStream || nodeResponse, {
                     status: statusCode,
                     statusText,
                     headers: nodeResponse.headers,
                     url: fetchRequest.url,
-                    signal,
                 });
                 resolve(ponyfillResponse);
             });

package/cjs/utils.js

@@ -7,7 +7,6 @@ exports.isArrayBufferView = isArrayBufferView;
 exports.isNodeReadable = isNodeReadable;
 exports.isIterable = isIterable;
 exports.shouldRedirect = shouldRedirect;
-exports.pipeThrough = pipeThrough;
 exports.endStream = endStream;
 exports.safeWrite = safeWrite;
 const node_events_1 = require("node:events");
@@ -49,42 +48,6 @@ function isIterable(value) {
 function shouldRedirect(status) {
     return status === 301 || status === 302 || status === 303 || status === 307 || status === 308;
 }
-function pipeThrough({ src, dest, signal, onError, }) {
-    if (onError) {
-        // listen for errors on the destination stream if necessary. if the readable
-        // stream (src) emits an error, the writable destination (dest) will be
-        // destroyed with that error (see below)
-        dest.once('error', onError);
-    }
-    src.once('error', (e) => {
-        // if the readable stream (src) emits an error during pipe, the writable
-        // destination (dest) is not closed automatically. that needs to be
-        // done manually. the readable stream is closed when error is emitted,
-        // so only the writable destination needs to be destroyed
-        dest.destroy(e);
-    });
-    if (signal) {
-        // this is faster than `import('node:signal').addAbortSignal(signal, src)`
-        const srcRef = new WeakRef(src);
-        const signalRef = new WeakRef(signal);
-        function cleanup() {
-            signalRef.deref()?.removeEventListener('abort', onAbort);
-            srcRef.deref()?.removeListener('end', cleanup);
-            srcRef.deref()?.removeListener('error', cleanup);
-            srcRef.deref()?.removeListener('close', cleanup);
-        }
-        function onAbort() {
-            srcRef.deref()?.destroy(new AbortError());
-            cleanup();
-        }
-        signal.addEventListener('abort', onAbort, { once: true });
-        // this is faster than `import('node:signal').finished(src, cleanup)`
-        src.once('end', cleanup);
-        src.once('error', cleanup);
-        src.once('close', cleanup);
-    }
-    src.pipe(dest, { end: true /* already default */ });
-}
 function endStream(stream) {
     // @ts-expect-error Avoid arguments adaptor trampoline https://v8.dev/blog/adaptor-frame
     return stream.end(null, null, null);
@@ -95,10 +58,3 @@ function safeWrite(chunk, stream) {
         return (0, node_events_1.once)(stream, 'drain');
     }
 }
-// https://github.com/nodejs/node/blob/f692878dec6354c0a82241f224906981861bc840/lib/internal/errors.js#L961-L973
-class AbortError extends Error {
-    constructor(message = 'The operation was aborted', options = undefined) {
-        super(message, options);
-        this.name = 'AbortError';
-    }
-}

package/esm/Body.js

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/ban-ts-comment */
 import { Buffer } from 'node:buffer';
-import { addAbortSignal, Readable } from 'node:stream';
+import { Readable } from 'node:stream';
 import { Busboy } from '@fastify/busboy';
 import { handleMaybePromise } from '@whatwg-node/promise-helpers';
 import { hasArrayBufferMethod, hasBufferMethod, hasBytesMethod, PonyfillBlob } from './Blob.js';
@@ -24,11 +24,9 @@ export class PonyfillBody {
     bodyUsed = false;
     contentType = null;
     contentLength = null;
-    _signal = null;
     constructor(bodyInit, options = {}) {
         this.bodyInit = bodyInit;
         this.options = options;
-        this._signal = options.signal || null;
         const { bodyFactory, contentType, contentLength, bodyType, buffer } = processBodyInit(bodyInit);
         this._bodyFactory = bodyFactory;
         this.contentType = contentType;
@@ -218,9 +216,6 @@ export class PonyfillBody {
                 limits: formDataLimits,
                 defCharset: 'utf-8',
             });
-            if (this._signal) {
-                addAbortSignal(this._signal, bb);
-            }
             let completed = false;
             const complete = (err) => {
                 if (completed)

package/esm/Request.js

@@ -73,6 +73,7 @@ export class PonyfillRequest extends PonyfillBody {
                 this.agent = requestInit.agent;
             }
         }
+        this._signal = requestInit?.signal || undefined;
     }
     headersSerializer;
     cache;
@@ -88,6 +89,7 @@ export class PonyfillRequest extends PonyfillBody {
     referrer;
     referrerPolicy;
     _url;
+    _signal;
     get signal() {
         this._signal ||= new AbortController().signal;
         return this._signal;

package/esm/fetchCurl.js

@@ -2,7 +2,7 @@ import { PassThrough, Readable } from 'node:stream';
 import { rootCertificates } from 'node:tls';
 import { createDeferredPromise } from '@whatwg-node/promise-helpers';
 import { PonyfillResponse } from './Response.js';
-import { defaultHeadersSerializer, isNodeReadable, pipeThrough, shouldRedirect } from './utils.js';
+import { defaultHeadersSerializer, isNodeReadable, shouldRedirect } from './utils.js';
 export function fetchCurl(fetchRequest) {
     const { Curl, CurlFeature, CurlPause, CurlProgressFunc } = globalThis['libcurl'];
     const curlHandle = new Curl();
@@ -19,7 +19,7 @@ export function fetchCurl(fetchRequest) {
     }
     curlHandle.enable(CurlFeature.StreamResponse);
     let signal;
-    if (fetchRequest._signal == null) {
+    if (fetchRequest._signal === null) {
         signal = undefined;
     }
     else if (fetchRequest._signal) {
@@ -101,12 +101,8 @@ export function fetchCurl(fetchRequest) {
         }
     });
     curlHandle.once('stream', function streamListener(stream, status, headersBuf) {
-        const passThrough = new PassThrough();
-        pipeThrough({
-            src: stream,
-            dest: passThrough,
-            signal,
-            onError: deferredPromise.reject,
+        const outputStream = stream.pipe(new PassThrough(), {
+            end: true,
         });
         const headersFlat = headersBuf
             .toString('utf8')
@@ -119,7 +115,7 @@ export function fetchCurl(fetchRequest) {
                     if (!stream.destroyed) {
                         stream.resume();
                     }
-                    passThrough.destroy();
+                    outputStream.destroy();
                     deferredPromise.reject(new Error('redirect is not allowed'));
                 }
                 return true;
@@ -127,14 +123,14 @@ export function fetchCurl(fetchRequest) {
             return false;
         });
         const headersInit = headersFlat.map(headerFlat => headerFlat.split(/:\s(.+)/).slice(0, 2));
-        const ponyfillResponse = new PonyfillResponse(passThrough, {
+        const ponyfillResponse = new PonyfillResponse(outputStream, {
             status,
             headers: headersInit,
             url: curlHandle.getInfo(Curl.info.REDIRECT_URL)?.toString() || fetchRequest.url,
             redirected: Number(curlHandle.getInfo(Curl.info.REDIRECT_COUNT)) > 0,
         });
         deferredPromise.resolve(ponyfillResponse);
-        streamResolved = passThrough;
+        streamResolved = outputStream;
     });
     setImmediate(() => {
         curlHandle.perform();

package/esm/fetchNodeHttp.js

@@ -6,7 +6,7 @@ import { handleMaybePromise } from '@whatwg-node/promise-helpers';
 import { PonyfillRequest } from './Request.js';
 import { PonyfillResponse } from './Response.js';
 import { PonyfillURL } from './URL.js';
-import { endStream, getHeadersObj, isNodeReadable, pipeThrough, safeWrite, shouldRedirect, } from './utils.js';
+import { endStream, getHeadersObj, isNodeReadable, safeWrite, shouldRedirect } from './utils.js';
 function getRequestFnForProtocol(url) {
     if (url.startsWith('http:')) {
         return httpRequest;
@@ -89,24 +89,20 @@ export function fetchNodeHttp(fetchRequest) {
                         return;
                     }
                 }
-                outputStream ||= new PassThrough();
-                pipeThrough({
-                    src: nodeResponse,
-                    dest: outputStream,
+                outputStream ||= new PassThrough({
                     signal,
-                    onError: reject,
                 });
+                nodeResponse.pipe(outputStream);
                 const statusCode = nodeResponse.statusCode || 200;
                 let statusText = nodeResponse.statusMessage || STATUS_CODES[statusCode];
                 if (statusText == null) {
                     statusText = '';
                 }
-                const ponyfillResponse = new PonyfillResponse(outputStream, {
+                const ponyfillResponse = new PonyfillResponse(outputStream || nodeResponse, {
                     status: statusCode,
                     statusText,
                     headers: nodeResponse.headers,
                     url: fetchRequest.url,
-                    signal,
                 });
                 resolve(ponyfillResponse);
             });

package/esm/utils.js

@@ -36,42 +36,6 @@ export function isIterable(value) {
 export function shouldRedirect(status) {
     return status === 301 || status === 302 || status === 303 || status === 307 || status === 308;
 }
-export function pipeThrough({ src, dest, signal, onError, }) {
-    if (onError) {
-        // listen for errors on the destination stream if necessary. if the readable
-        // stream (src) emits an error, the writable destination (dest) will be
-        // destroyed with that error (see below)
-        dest.once('error', onError);
-    }
-    src.once('error', (e) => {
-        // if the readable stream (src) emits an error during pipe, the writable
-        // destination (dest) is not closed automatically. that needs to be
-        // done manually. the readable stream is closed when error is emitted,
-        // so only the writable destination needs to be destroyed
-        dest.destroy(e);
-    });
-    if (signal) {
-        // this is faster than `import('node:signal').addAbortSignal(signal, src)`
-        const srcRef = new WeakRef(src);
-        const signalRef = new WeakRef(signal);
-        function cleanup() {
-            signalRef.deref()?.removeEventListener('abort', onAbort);
-            srcRef.deref()?.removeListener('end', cleanup);
-            srcRef.deref()?.removeListener('error', cleanup);
-            srcRef.deref()?.removeListener('close', cleanup);
-        }
-        function onAbort() {
-            srcRef.deref()?.destroy(new AbortError());
-            cleanup();
-        }
-        signal.addEventListener('abort', onAbort, { once: true });
-        // this is faster than `import('node:signal').finished(src, cleanup)`
-        src.once('end', cleanup);
-        src.once('error', cleanup);
-        src.once('close', cleanup);
-    }
-    src.pipe(dest, { end: true /* already default */ });
-}
 export function endStream(stream) {
     // @ts-expect-error Avoid arguments adaptor trampoline https://v8.dev/blog/adaptor-frame
     return stream.end(null, null, null);
@@ -82,10 +46,3 @@ export function safeWrite(chunk, stream) {
         return once(stream, 'drain');
     }
 }
-// https://github.com/nodejs/node/blob/f692878dec6354c0a82241f224906981861bc840/lib/internal/errors.js#L961-L973
-class AbortError extends Error {
-    constructor(message = 'The operation was aborted', options = undefined) {
-        super(message, options);
-        this.name = 'AbortError';
-    }
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@whatwg-node/node-fetch",
-  "version": "0.7.23-alpha-20250726023513-1ae33d44a98b7e7238a179b7b100c6bdb1de96e7",
+  "version": "0.7.23-alpha-20250726084802-683ac7e676415a5d08903fcace71146b2a74a837",
   "description": "Fetch API implementation for Node",
   "sideEffects": false,
   "dependencies": {

package/typings/Body.d.cts

@@ -24,7 +24,6 @@ export declare class PonyfillBody<TJSON = any> implements Body {
     bodyUsed: boolean;
     contentType: string | null;
     contentLength: number | null;
-    _signal?: AbortSignal | null;
     constructor(bodyInit: BodyPonyfillInit | null, options?: PonyfillBodyOptions);
     private bodyType?;
     private _bodyFactory;

package/typings/Body.d.ts

@@ -24,7 +24,6 @@ export declare class PonyfillBody<TJSON = any> implements Body {
     bodyUsed: boolean;
     contentType: string | null;
     contentLength: number | null;
-    _signal?: AbortSignal | null;
     constructor(bodyInit: BodyPonyfillInit | null, options?: PonyfillBodyOptions);
     private bodyType?;
     private _bodyFactory;

package/typings/Request.d.cts

@@ -26,6 +26,7 @@ export declare class PonyfillRequest<TJSON = any> extends PonyfillBody<TJSON> im
     referrer: string;
     referrerPolicy: ReferrerPolicy;
     _url: string | undefined;
+    _signal: AbortSignal | undefined;
     get signal(): AbortSignal;
     get url(): string;
     _parsedUrl: URL | undefined;

package/typings/Request.d.ts

@@ -26,6 +26,7 @@ export declare class PonyfillRequest<TJSON = any> extends PonyfillBody<TJSON> im
     referrer: string;
     referrerPolicy: ReferrerPolicy;
     _url: string | undefined;
+    _signal: AbortSignal | undefined;
     get signal(): AbortSignal;
     get url(): string;
     _parsedUrl: URL | undefined;

package/typings/utils.d.cts

@@ -6,12 +6,6 @@ export declare function isArrayBufferView(obj: any): obj is ArrayBufferView;
 export declare function isNodeReadable(obj: any): obj is Readable;
 export declare function isIterable(value: any): value is Iterable<unknown>;
 export declare function shouldRedirect(status?: number): boolean;
-export declare function pipeThrough({ src, dest, signal, onError, }: {
-    src: Readable;
-    dest: Writable;
-    signal?: AbortSignal | undefined;
-    onError?: ((e: Error) => void) | undefined;
-}): void;
 export declare function endStream(stream: {
     end: () => void;
 }): void;

package/typings/utils.d.ts

@@ -6,12 +6,6 @@ export declare function isArrayBufferView(obj: any): obj is ArrayBufferView;
 export declare function isNodeReadable(obj: any): obj is Readable;
 export declare function isIterable(value: any): value is Iterable<unknown>;
 export declare function shouldRedirect(status?: number): boolean;
-export declare function pipeThrough({ src, dest, signal, onError, }: {
-    src: Readable;
-    dest: Writable;
-    signal?: AbortSignal | undefined;
-    onError?: ((e: Error) => void) | undefined;
-}): void;
 export declare function endStream(stream: {
     end: () => void;
 }): void;