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

[GHSA-jr5f-v2jv-69x6] axios Requests Vulnerable To Possible SSRF and Credential Leakage via Absolute URL #5356

Open
wants to merge 1 commit into
base: mhassan1/advisory-improvement-5356
Choose a base branch
from

Conversation

mhassan1
Copy link

Updates

  • Affected products
  • CVSS v4
  • References

Comments
Version 1.8.2 contained an incomplete fix (it only included the http adapter); version 1.8.3 contains the fix for the xhr and fetch adapters

@github
Copy link
Collaborator

github commented Mar 12, 2025

Hi there @jasonsaayman! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions github-actions bot changed the base branch from main to mhassan1/advisory-improvement-5356 March 12, 2025 11:38
@mhassan1 mhassan1 marked this pull request as draft March 12, 2025 11:46
@@ -11,7 +11,7 @@
"severity": [
{
"type": "CVSS_V4",
"score": "CVSS:4.0/AV:N/AC:L/AT:N/PR:N/UI:N/VC:H/VI:N/VA:N/SC:N/SI:N/SA:N/E:P"
Copy link
Author

Choose a reason for hiding this comment

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

I didn't intend to remove the E metric, but the advisory improvement screen forced me to remove it, and I don't see a way to put it back.

Copy link
Author

Choose a reason for hiding this comment

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

I've opened an issue about this limitation in the advisory improvement screen: #5357.

@JonathanLEvans
Copy link

If there is an incomplete fix, then a separate advisory and CVE ID may be issued. I would like @jasonsaayman's thought on how to handle this.

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

Successfully merging this pull request may close these issues.

3 participants