chore(deps): update dependency follow-redirects to v1.16.0 [security]#16
Open
renovate[bot] wants to merge 1 commit into
Open
chore(deps): update dependency follow-redirects to v1.16.0 [security]#16renovate[bot] wants to merge 1 commit into
renovate[bot] wants to merge 1 commit into
Conversation
6fee892 to
1e8ef87
Compare
2a9c68b to
e56ee74
Compare
e56ee74 to
76b3e4b
Compare
76b3e4b to
2b9353f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR contains the following updates:
1.15.1→1.16.0Follow Redirects improperly handles URLs in the url.parse() function
CVE-2023-26159 / GHSA-jchw-25xp-jwwc
More information
Details
Versions of the package follow-redirects before 1.15.4 are vulnerable to Improper Input Validation due to the improper handling of URLs by the url.parse() function. When new URL() throws an error, it can be manipulated to misinterpret the hostname. An attacker could exploit this weakness to redirect traffic to a malicious site, potentially leading to information disclosure, phishing attacks, or other security breaches.
Severity
CVSS:3.1/AV:N/AC:L/PR:N/UI:R/S:C/C:L/I:L/A:NReferences
This data is provided by the GitHub Advisory Database (CC-BY 4.0).
follow-redirects' Proxy-Authorization header kept across hosts
CVE-2024-28849 / GHSA-cxjh-pqwp-8mfp
More information
Details
When using axios, its dependency follow-redirects only clears authorization header during cross-domain redirect, but allows the proxy-authentication header which contains credentials too.
Steps To Reproduce & PoC
Test code:
When I meet the cross-domain redirect, the sensitive headers like authorization and cookie are cleared, but proxy-authentication header is kept.
Impact
This vulnerability may lead to credentials leak.
Recommendations
Remove proxy-authentication header during cross-domain redirect
Recommended Patch
follow-redirects/index.js:464
Severity
CVSS:3.1/AV:N/AC:L/PR:L/UI:N/S:U/C:H/I:N/A:NReferences
This data is provided by the GitHub Advisory Database (CC-BY 4.0).
follow-redirects leaks Custom Authentication Headers to Cross-Domain Redirect Targets
GHSA-r4q5-vmmm-2653
More information
Details
Summary
When an HTTP request follows a cross-domain redirect (301/302/307/308),
follow-redirectsonly stripsauthorization,proxy-authorization, andcookieheaders (matched by regex at index.js:469-476). Any custom authentication header (e.g.,X-API-Key,X-Auth-Token,Api-Key,Token) is forwarded verbatim to the redirect target.Since
follow-redirectsis the redirect-handling dependency for axios (105K+ stars), this vulnerability affects the entire axios ecosystem.Affected Code
index.js, lines 469-476:The regex only matches
authorization,proxy-authorization, andcookie. Custom headers likeX-API-Keyare not matched.Attack Scenario
headers: { 'X-API-Key': 'sk-live-secret123' }302 Location: https://evil.com/stealX-API-Key: sk-live-secret123toevil.comImpact
Any custom auth header set via axios leaks on cross-domain redirect. Extremely common pattern. Affects all axios users in Node.js.
Suggested Fix
Add a
sensitiveHeadersoption that users can extend, or strip ALL non-standard headers on cross-domain redirect.Disclosure
Source code review, manually verified. Found 2026-03-20.
Severity
CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:L/VI:N/VA:N/SC:N/SI:N/SA:NReferences
This data is provided by the GitHub Advisory Database (CC-BY 4.0).
Release Notes
follow-redirects/follow-redirects (follow-redirects)
v1.16.0Compare Source
v1.15.11Compare Source
v1.15.10Compare Source
v1.15.9Compare Source
v1.15.8Compare Source
v1.15.7Compare Source
v1.15.6Compare Source
v1.15.5Compare Source
v1.15.4Compare Source
v1.15.3Compare Source
v1.15.2Compare Source
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.