@whatwg-node/node-fetch 0.7.9-alpha-20250213122700-6a189724db9ab6211349779c44665b0b4f1006af → 0.7.9-rc-20250213120211-2e564250f0d6354be9db6c60c01427b07608eed5

package/cjs/Headers.js

@@ -247,7 +247,7 @@ class PonyfillHeaders {
                 record['set-cookie'] = this._setCookies;
             }
             else {
-                record[key] = value?.includes(',') ? value.split(',').map(el => el.trim()) : value;
+                record[key] = value.includes(',') ? value.split(',').map(el => el.trim()) : value;
             }
         });
         return `Headers ${(0, node_util_1.inspect)(record)}`;

package/cjs/fetchCurl.js

@@ -118,8 +118,7 @@ function fetchCurl(fetchRequest) {
             .filter(headerFilter => {
             if (headerFilter && !headerFilter.startsWith('HTTP/')) {
                 if (fetchRequest.redirect === 'error' &&
-                    headerFilter.toLowerCase().includes('location') &&
-                    (0, utils_js_1.shouldRedirect)(status)) {
+                    (headerFilter.includes('location') || headerFilter.includes('Location'))) {
                     if (!stream.destroyed) {
                         stream.resume();
                     }
@@ -151,14 +150,7 @@ function fetchCurl(fetchRequest) {
         });
     }
     else {
-        try {
-            curlHandle.perform();
-        }
-        catch (e) {
-            setImmediate(() => {
-                curlHandle.perform();
-            });
-        }
+        curlHandle.perform();
     }
     return deferredPromise.promise;
 }

package/cjs/fetchNodeHttp.js

@@ -73,7 +73,7 @@ function fetchNodeHttp(fetchRequest) {
                     default:
                         outputStream = new node_stream_1.PassThrough();
                 }
-                if (nodeResponse.headers.location && (0, utils_js_1.shouldRedirect)(nodeResponse.statusCode)) {
+                if (nodeResponse.headers.location) {
                     if (fetchRequest.redirect === 'error') {
                         const redirectError = new Error('Redirects are not allowed');
                         reject(redirectError);

package/cjs/utils.js

@@ -7,7 +7,6 @@ exports.isArrayBufferView = isArrayBufferView;
 exports.isNodeReadable = isNodeReadable;
 exports.createDeferredPromise = createDeferredPromise;
 exports.isIterable = isIterable;
-exports.shouldRedirect = shouldRedirect;
 function isHeadersInstance(obj) {
     return obj?.forEach != null;
 }
@@ -89,6 +88,3 @@ function createDeferredPromise() {
 function isIterable(value) {
     return value?.[Symbol.iterator] != null;
 }
-function shouldRedirect(status) {
-    return status === 301 || status === 302 || status === 303 || status === 307 || status === 308;
-}

package/esm/Headers.js

@@ -243,7 +243,7 @@ export class PonyfillHeaders {
                 record['set-cookie'] = this._setCookies;
             }
             else {
-                record[key] = value?.includes(',') ? value.split(',').map(el => el.trim()) : value;
+                record[key] = value.includes(',') ? value.split(',').map(el => el.trim()) : value;
             }
         });
         return `Headers ${inspect(record)}`;

package/esm/fetchCurl.js

@@ -2,7 +2,7 @@ import { PassThrough, Readable } from 'node:stream';
 import { pipeline } from 'node:stream/promises';
 import { rootCertificates } from 'node:tls';
 import { PonyfillResponse } from './Response.js';
-import { createDeferredPromise, defaultHeadersSerializer, isNodeReadable, shouldRedirect, } from './utils.js';
+import { createDeferredPromise, defaultHeadersSerializer, isNodeReadable } from './utils.js';
 export function fetchCurl(fetchRequest) {
     const { Curl, CurlFeature, CurlPause, CurlProgressFunc } = globalThis['libcurl'];
     const curlHandle = new Curl();
@@ -115,8 +115,7 @@ export function fetchCurl(fetchRequest) {
             .filter(headerFilter => {
             if (headerFilter && !headerFilter.startsWith('HTTP/')) {
                 if (fetchRequest.redirect === 'error' &&
-                    headerFilter.toLowerCase().includes('location') &&
-                    shouldRedirect(status)) {
+                    (headerFilter.includes('location') || headerFilter.includes('Location'))) {
                     if (!stream.destroyed) {
                         stream.resume();
                     }
@@ -148,14 +147,7 @@ export function fetchCurl(fetchRequest) {
         });
     }
     else {
-        try {
-            curlHandle.perform();
-        }
-        catch (e) {
-            setImmediate(() => {
-                curlHandle.perform();
-            });
-        }
+        curlHandle.perform();
     }
     return deferredPromise.promise;
 }

package/esm/fetchNodeHttp.js

@@ -6,7 +6,7 @@ import { createBrotliDecompress, createGunzip, createInflate, createInflateRaw }
 import { PonyfillRequest } from './Request.js';
 import { PonyfillResponse } from './Response.js';
 import { PonyfillURL } from './URL.js';
-import { getHeadersObj, isNodeReadable, shouldRedirect } from './utils.js';
+import { getHeadersObj, isNodeReadable } from './utils.js';
 function getRequestFnForProtocol(url) {
     if (url.startsWith('http:')) {
         return httpRequest;
@@ -70,7 +70,7 @@ export function fetchNodeHttp(fetchRequest) {
                     default:
                         outputStream = new PassThrough();
                 }
-                if (nodeResponse.headers.location && shouldRedirect(nodeResponse.statusCode)) {
+                if (nodeResponse.headers.location) {
                     if (fetchRequest.redirect === 'error') {
                         const redirectError = new Error('Redirects are not allowed');
                         reject(redirectError);

package/esm/utils.js

@@ -79,6 +79,3 @@ export function createDeferredPromise() {
 export function isIterable(value) {
     return value?.[Symbol.iterator] != null;
 }
-export function shouldRedirect(status) {
-    return status === 301 || status === 302 || status === 303 || status === 307 || status === 308;
-}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@whatwg-node/node-fetch",
-  "version": "0.7.9-alpha-20250213122700-6a189724db9ab6211349779c44665b0b4f1006af",
+  "version": "0.7.9-rc-20250213120211-2e564250f0d6354be9db6c60c01427b07608eed5",
   "description": "Fetch API implementation for Node",
   "sideEffects": false,
   "dependencies": {

package/typings/utils.d.cts

@@ -11,4 +11,3 @@ export interface DeferredPromise<T = void> {
 }
 export declare function createDeferredPromise<T = void>(): DeferredPromise<T>;
 export declare function isIterable(value: any): value is Iterable<unknown>;
-export declare function shouldRedirect(status?: number): boolean;

package/typings/utils.d.ts

@@ -11,4 +11,3 @@ export interface DeferredPromise<T = void> {
 }
 export declare function createDeferredPromise<T = void>(): DeferredPromise<T>;
 export declare function isIterable(value: any): value is Iterable<unknown>;
-export declare function shouldRedirect(status?: number): boolean;