Skip to content

fix: fixing redirect issue with node v24#233

Merged
leftieFriele merged 1 commit into
mainfrom
redirect-bugs
May 7, 2025
Merged

fix: fixing redirect issue with node v24#233
leftieFriele merged 1 commit into
mainfrom
redirect-bugs

Conversation

@leftieFriele
Copy link
Copy Markdown
Contributor

In Node.js v24 Undici was updated to version 7, which included dropping support for the default 'maxRedirections' option in undici.request, leading redirects to return empty responses.

This fix uses an interceptor to enable redirects.

In Node.js v24 Undici was updated to version 7, which included dropping
support for the default 'maxRedirections' option in undici.request, leading redirects to return
empty responses.

This fix uses an interceptor to enable redirects.

This fix also adds a configuration option for maximum number redirects.
Copy link
Copy Markdown
Contributor

@trygve-lie trygve-lie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Well spotted.

@leftieFriele leftieFriele merged commit 8bc43ca into main May 7, 2025
12 checks passed
@leftieFriele leftieFriele deleted the redirect-bugs branch May 7, 2025 14:55
github-actions Bot pushed a commit that referenced this pull request May 7, 2025
## [4.0.68](v4.0.67...v4.0.68) (2025-05-07)

### Bug Fixes

* fixing redirect issue with node v24 ([#233](#233)) ([8bc43ca](8bc43ca))
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2025

🎉 This PR is included in version 4.0.68 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants