@whatwg-node/node-fetch 0.7.23-alpha-20250725201224-a3738367f9a514964b76715b19d4bb76c0bc0411 → 0.7.23-alpha-20250726003115-27dea3e7c5dae7285bf9c837cce54f387e269df2

package/cjs/Body.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.PonyfillBody = void 0;
 /* eslint-disable @typescript-eslint/ban-ts-comment */
 const node_buffer_1 = require("node:buffer");
-const node_http_1 = require("node:http");
 const node_stream_1 = require("node:stream");
 const busboy_1 = require("@fastify/busboy");
 const promise_helpers_1 = require("@whatwg-node/promise-helpers");
@@ -28,12 +27,10 @@ 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, options?.signal);
+        const { bodyFactory, contentType, contentLength, bodyType, buffer } = processBodyInit(bodyInit);
         this._bodyFactory = bodyFactory;
         this.contentType = contentType;
         this.contentLength = contentLength;
@@ -218,9 +215,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)
@@ -373,7 +367,7 @@ class PonyfillBody {
     }
 }
 exports.PonyfillBody = PonyfillBody;
-function processBodyInit(bodyInit, signal) {
+function processBodyInit(bodyInit) {
     if (bodyInit == null) {
         return {
             bodyFactory: () => null,
@@ -456,22 +450,6 @@ function processBodyInit(bodyInit, signal) {
             },
         };
     }
-    if (bodyInit instanceof node_http_1.IncomingMessage) {
-        const passThrough = new node_stream_1.PassThrough();
-        (0, utils_js_1.pipeThrough)({
-            src: bodyInit,
-            dest: passThrough,
-            signal,
-        });
-        return {
-            bodyType: BodyInitType.Readable,
-            contentType: null,
-            contentLength: null,
-            bodyFactory() {
-                return new ReadableStream_js_1.PonyfillReadableStream(passThrough);
-            },
-        };
-    }
     if (bodyInit instanceof node_stream_1.Readable) {
         return {
             bodyType: BodyInitType.Readable,

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

@@ -28,9 +28,6 @@ function fetchCurl(fetchRequest) {
     else if (fetchRequest._signal) {
         signal = fetchRequest._signal;
     }
-    else {
-        signal = fetchRequest.signal;
-    }
     curlHandle.setStreamProgressCallback(function () {
         return signal?.aborted ? (process.env.DEBUG ? CurlProgressFunc.Continue : 1) : 0;
     });
@@ -107,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')
@@ -125,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;
@@ -133,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

@@ -29,15 +29,12 @@ function fetchNodeHttp(fetchRequest) {
                 nodeHeaders['accept-encoding'] = 'gzip, deflate, br';
             }
             let signal;
-            if (fetchRequest._signal === null) {
+            if (fetchRequest._signal == null) {
                 signal = undefined;
             }
             else if (fetchRequest._signal) {
                 signal = fetchRequest._signal;
             }
-            else {
-                signal = fetchRequest.signal;
-            }
             let nodeRequest;
             // If it is our ponyfilled Request, it should have `parsedUrl` which is a `URL` object
             if (fetchRequest.parsedUrl) {
@@ -95,14 +92,10 @@ function fetchNodeHttp(fetchRequest) {
                         return;
                     }
                 }
-                if (outputStream != null) {
-                    (0, utils_js_1.pipeThrough)({
-                        src: nodeResponse,
-                        dest: outputStream,
-                        signal,
-                        onError: reject,
-                    });
-                }
+                outputStream ||= new node_stream_1.PassThrough();
+                nodeResponse.pipe(outputStream, {
+                    end: true,
+                });
                 const statusCode = nodeResponse.statusCode || 200;
                 let statusText = nodeResponse.statusMessage || node_http_1.STATUS_CODES[statusCode];
                 if (statusText == null) {

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,14 +1,13 @@
 /* eslint-disable @typescript-eslint/ban-ts-comment */
 import { Buffer } from 'node:buffer';
-import { IncomingMessage } from 'node:http';
-import { addAbortSignal, PassThrough, 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';
 import { PonyfillFile } from './File.js';
 import { getStreamFromFormData, PonyfillFormData } from './FormData.js';
 import { PonyfillReadableStream } from './ReadableStream.js';
-import { fakePromise, isArrayBufferView, pipeThrough } from './utils.js';
+import { fakePromise, isArrayBufferView } from './utils.js';
 var BodyInitType;
 (function (BodyInitType) {
     BodyInitType["ReadableStream"] = "ReadableStream";
@@ -25,12 +24,10 @@ 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, options?.signal);
+        const { bodyFactory, contentType, contentLength, bodyType, buffer } = processBodyInit(bodyInit);
         this._bodyFactory = bodyFactory;
         this.contentType = contentType;
         this.contentLength = contentLength;
@@ -215,9 +212,6 @@ export class PonyfillBody {
                 limits: formDataLimits,
                 defCharset: 'utf-8',
             });
-            if (this._signal) {
-                addAbortSignal(this._signal, bb);
-            }
             let completed = false;
             const complete = (err) => {
                 if (completed)
@@ -369,7 +363,7 @@ export class PonyfillBody {
         });
     }
 }
-function processBodyInit(bodyInit, signal) {
+function processBodyInit(bodyInit) {
     if (bodyInit == null) {
         return {
             bodyFactory: () => null,
@@ -452,22 +446,6 @@ function processBodyInit(bodyInit, signal) {
             },
         };
     }
-    if (bodyInit instanceof IncomingMessage) {
-        const passThrough = new PassThrough();
-        pipeThrough({
-            src: bodyInit,
-            dest: passThrough,
-            signal,
-        });
-        return {
-            bodyType: BodyInitType.Readable,
-            contentType: null,
-            contentLength: null,
-            bodyFactory() {
-                return new PonyfillReadableStream(passThrough);
-            },
-        };
-    }
     if (bodyInit instanceof Readable) {
         return {
             bodyType: BodyInitType.Readable,

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();
@@ -25,9 +25,6 @@ export function fetchCurl(fetchRequest) {
     else if (fetchRequest._signal) {
         signal = fetchRequest._signal;
     }
-    else {
-        signal = fetchRequest.signal;
-    }
     curlHandle.setStreamProgressCallback(function () {
         return signal?.aborted ? (process.env.DEBUG ? CurlProgressFunc.Continue : 1) : 0;
     });
@@ -104,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')
@@ -122,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;
@@ -130,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

@@ -1,12 +1,12 @@
 import { request as httpRequest, STATUS_CODES } from 'node:http';
 import { request as httpsRequest } from 'node:https';
-import { Readable } from 'node:stream';
+import { PassThrough, Readable } from 'node:stream';
 import { createBrotliDecompress, createGunzip, createInflate, createInflateRaw } from 'node:zlib';
 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;
@@ -26,15 +26,12 @@ export function fetchNodeHttp(fetchRequest) {
                 nodeHeaders['accept-encoding'] = 'gzip, deflate, br';
             }
             let signal;
-            if (fetchRequest._signal === null) {
+            if (fetchRequest._signal == null) {
                 signal = undefined;
             }
             else if (fetchRequest._signal) {
                 signal = fetchRequest._signal;
             }
-            else {
-                signal = fetchRequest.signal;
-            }
             let nodeRequest;
             // If it is our ponyfilled Request, it should have `parsedUrl` which is a `URL` object
             if (fetchRequest.parsedUrl) {
@@ -92,14 +89,10 @@ export function fetchNodeHttp(fetchRequest) {
                         return;
                     }
                 }
-                if (outputStream != null) {
-                    pipeThrough({
-                        src: nodeResponse,
-                        dest: outputStream,
-                        signal,
-                        onError: reject,
-                    });
-                }
+                outputStream ||= new PassThrough();
+                nodeResponse.pipe(outputStream, {
+                    end: true,
+                });
                 const statusCode = nodeResponse.statusCode || 200;
                 let statusText = nodeResponse.statusMessage || STATUS_CODES[statusCode];
                 if (statusText == null) {

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-20250725201224-a3738367f9a514964b76715b19d4bb76c0bc0411",
+  "version": "0.7.23-alpha-20250726003115-27dea3e7c5dae7285bf9c837cce54f387e269df2",
   "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;