[Aikido] Fix security issue in axios via minor version upgrade from 1.13.4 to 1.19.0 - #168
Open
aikido-autofix[bot] wants to merge 1 commit into
Open
Conversation
|
🚨 gitStream Monthly Automation Limit Reached 🚨 Your organization has exceeded the number of pull requests allowed for automation with gitStream. To continue automating your PR workflows and unlock additional features, please contact LinearB. |
✅ Security Analysis ResultsGreat news! No security issues found in this pull request. Analysis Summary:
💡 Trigger a new security scan by commenting Security analysis powered by Claude Sonnet 4.6 via pr-auditor | Questions? Contact #dx-team or check out this page |
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.
Summary / Description
Upgrade axios to fix critical security vulnerabilities including proxy bypass/SSRF, prototype pollution MITM attacks, credential header leakage on redirects, and sensitive data exposure in error serialization.
Security Impact — CVE vulnerabilities fixed by this PR
✅ 18 CVEs resolved by this upgrade
This PR will resolve the following CVEs:
X-API-Keyand AWS tokens to unintended hosts, allowing attackers to steal sensitive authentication data. This information disclosure vulnerability affects shared environments where secret headers are set by default.formDataToJSONwhen processing deeply nested FormData field names causes stack overflow, resulting in denial of service for applications converting untrusted FormData to JSON.Object.prototypeproperties to inject maliciousAuthorizationheaders and manipulate query strings when applications pass placeholder objects likeauth: {}orparamsSerializer: {}. This enables credential exfiltration and request tampering when another component has already polluted the prototype chain.maxBodyLengthlimits on WHATWGReadableStreamrequest bodies with unknown sizes, allowing attackers to bypass upload size restrictions and cause unexpected network egress or resource exhaustion.formDataToJSONallows attackers to crash the process by submitting FormData with deeply nested bracket-delimited field names, causing stack exhaustion and denial of service.//(e.g.,https:internal.example) were silently normalized instead of rejected, allowing attackers to bypass URL allowlists or WAF checks and reach unintended hosts. The fix now throws an error for invalid scheme URLs before normalization.formDataToJSONhelper allows attackers to polluteObject.prototypethrough specially crafted multipart field names, potentially enabling remote code execution or other malicious actions.maxBodyLengthenforcement when sent over HTTP/2, allowing attackers to transmit arbitrarily large payloads and exhaust resources or bypass configured egress limits. This affects Node.js applications forwarding untrusted streams to HTTP/2 endpoints.Breaking Changes & Upgrade Impact — prefer Warnings / Cautions /⚠️ sections over implementation sections (upgrade risks, breaking changes, manual migration steps)
Fix Details / Technical Implementation (detailed write-up of what was changed and why)
🤖 Remediation details
Bump
axiosto resolve multiple high/medium/low severity vulnerabilitiesShort summary
This PR remediates 18 CVEs and advisories affecting
axios, a direct runtime dependency declared in the rootpackage.json. The fix updates the declared version range inpackage.jsonand refreshesyarn.lockso the resolved version moves from the vulnerable1.13.4to1.19.0.axios
axiosis a direct dependency in the rootpackage.jsonunderdependencies. The declared range was^1.13.4, which resolved to1.13.4—a version affected by 18 advisories spanning SSRF, header injection, credential leakage, and related issues. The range was narrowed to^1.18.0(the minimum that satisfies all patched-version specs across the full advisory set), andyarn installresolved it to1.19.0, the latest compatible release. No parent bumps, transitive walks, orresolutionsoverrides were required becauseaxiosis already a direct dependency.Version changes
axios^1.13.4(resolved1.13.4)^1.18.0(resolved1.19.0)