-
Notifications
You must be signed in to change notification settings - Fork 78
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
fix: update axios to 1.6.1 to fix CVE-2023-45857 #147
Conversation
This should be merged sooner rather than later |
Thank you! |
@jeffbski Can this get some attention please? Thanks a lot. For more context: CVE-2023-45857 (CWE-359) XSRF-TOKEN value is disclosed to an unauthorised actor, fixed in axios 1.6.0. |
And update and a subsequent release would be great! |
package.json
Outdated
@@ -38,7 +38,7 @@ | |||
"temp": "^0.9.4" | |||
}, | |||
"dependencies": { | |||
"axios": "^0.27.2", | |||
"axios": "^1.6.0", |
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.
You can directly upgrade to 1.6.1.
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.
@porschesstein Thanks, updated. When I originally opened this PR, the latest version of Axios was still 1.6.0.
Guess we'll just have to wait-on this PR. |
Tested locally and worked perfectly 🚀 |
I hope this gets merged soon and then released! Looking forward... |
Thanks @AndrewMax for the PR and for those that confirmed it. It is published to wait-on@7.2.0 |
Summary: As per title. This avoids the axios vulnerability thanks to jeffbski/wait-on#147. Reviewed By: alanz Differential Revision: D51544987 fbshipit-source-id: f15a001aa1eaab14935cf521dd26dfa3938489e6
Axios was recently updated to fix CVE-2023-45857. This PR is to update to axios 1.6.0.