@whatwg-node/node-fetch 0.7.8 → 0.7.9-alpha-20250213113435-26995d499d83d94954d7335ea7f9963e568d5c57

package/cjs/fetchCurl.js

@@ -118,7 +118,8 @@ function fetchCurl(fetchRequest) {
             .filter(headerFilter => {
             if (headerFilter && !headerFilter.startsWith('HTTP/')) {
                 if (fetchRequest.redirect === 'error' &&
-                    (headerFilter.includes('location') || headerFilter.includes('Location'))) {
+                    (headerFilter.includes('location') || headerFilter.includes('Location')) &&
+                    (0, utils_js_1.shouldRedirect)(status)) {
                     if (!stream.destroyed) {
                         stream.resume();
                     }

package/cjs/fetchNodeHttp.js

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

package/cjs/utils.js

@@ -7,6 +7,7 @@ exports.isArrayBufferView = isArrayBufferView;
 exports.isNodeReadable = isNodeReadable;
 exports.createDeferredPromise = createDeferredPromise;
 exports.isIterable = isIterable;
+exports.shouldRedirect = shouldRedirect;
 function isHeadersInstance(obj) {
     return obj?.forEach != null;
 }
@@ -88,3 +89,6 @@ 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/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 } from './utils.js';
+import { createDeferredPromise, defaultHeadersSerializer, isNodeReadable, shouldRedirect, } from './utils.js';
 export function fetchCurl(fetchRequest) {
     const { Curl, CurlFeature, CurlPause, CurlProgressFunc } = globalThis['libcurl'];
     const curlHandle = new Curl();
@@ -115,7 +115,8 @@ export function fetchCurl(fetchRequest) {
             .filter(headerFilter => {
             if (headerFilter && !headerFilter.startsWith('HTTP/')) {
                 if (fetchRequest.redirect === 'error' &&
-                    (headerFilter.includes('location') || headerFilter.includes('Location'))) {
+                    (headerFilter.includes('location') || headerFilter.includes('Location')) &&
+                    shouldRedirect(status)) {
                     if (!stream.destroyed) {
                         stream.resume();
                     }

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 } from './utils.js';
+import { getHeadersObj, isNodeReadable, shouldRedirect } 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) {
+                if (nodeResponse.headers.location && shouldRedirect(nodeResponse.statusCode)) {
                     if (fetchRequest.redirect === 'error') {
                         const redirectError = new Error('Redirects are not allowed');
                         reject(redirectError);

package/esm/utils.js

@@ -79,3 +79,6 @@ 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.8",
+  "version": "0.7.9-alpha-20250213113435-26995d499d83d94954d7335ea7f9963e568d5c57",
   "description": "Fetch API implementation for Node",
   "sideEffects": false,
   "dependencies": {

package/typings/utils.d.cts

@@ -11,3 +11,4 @@ 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,3 +11,4 @@ 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;