-
-
Notifications
You must be signed in to change notification settings - Fork 11k
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
Hotfix: Prevent SSRF #3410
Hotfix: Prevent SSRF #3410
Conversation
Is |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@timemachine3030 Shared some suggestions with you. Thanks for your quick fixing.
@timemachine3030 Can you say if this only affected 0.21.0 or prior versions as well? |
@ArSn Versions since 0.19.0, when proxy forwarding was added. The vulnerability is exclusive to Node.js applications making requests through proxy servers. |
Yeah I caught that, thanks! |
Sorry if this is not the right place to ask, but when is the fixed code going to be packaged and published on the NPM repo, so we can actually make use of it? When is the next npm version supposed to come out? |
Any update on when |
@jasonsaayman When will 0.21.1 tag be released ?Thanks |
@twistedpair, @kobe0730 I have asked Emily to get to this release so it is with her now, I will ask her if she will get to it this week and revert back. Thanks for your patience. |
|
@jasonsaayman @emilyemorehouse Sorry to ping but any updates on when we can expect v0.21.1 will be released? Given the CVSS score on this I'm about to breach security SLOs (and judging by the interest from others I am not the only one). Thanks! |
Hello, same issue with my company - could we please release soon ? It will be a nice christmas gift 🙏 |
23480160548544 |
Fixes vulnerability described in:
Uses a hook in
follow-redirects
to continue using the proxy if a redirect is encountered.