Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Please fix version of Library "follow-redirects" to eliminate CVE-2023-26159 issue #1657

Open
scarton opened this issue Jan 16, 2024 · 3 comments · May be fixed by #1658 or #1660
Open

Please fix version of Library "follow-redirects" to eliminate CVE-2023-26159 issue #1657

scarton opened this issue Jan 16, 2024 · 3 comments · May be fixed by #1658 or #1660

Comments

@scarton
Copy link

scarton commented Jan 16, 2024

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.

Thank you.

@ryanomor
Copy link

ryanomor commented Jan 19, 2024

@indexzero @cronopio @jcrugzz @yawnt Any updates on if this is being addressed (#1571)?

@scarton
Copy link
Author

scarton commented Jan 19, 2024

FYI - I got around this for now by overriding the follow-redirects library version in my package.json:

"overrides": {
"http-server": {
"http-proxy": {
"follow-redirects": "^1.15.5"
}
}
},

@kanticsurapaneni
Copy link

Hello, The issue still seems to exist. Latest version of http-proxy (1.18.1) is still referencing old version of follow-redirects 1.0.0. Snyk is still reporting this vulnerability because of the reference to follow-redirects 1.0.0 version in package-lock.json being generated. Can you please take a look at this and fix the package.json for http-proxy module?
Reference: https://github.com/http-party/node-http-proxy/blob/master/package.json
npm-issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants