[Aikido] Fix 32 security issues in axios, form-data - #40
Open
aikido-autofix[bot] wants to merge 1 commit into
Open
[Aikido] Fix 32 security issues in axios, form-data#40aikido-autofix[bot] wants to merge 1 commit into
aikido-autofix[bot] wants to merge 1 commit into
Conversation
6 tasks
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.
Problem
Upgrade axios and form-data to fix critical SSRF, proxy bypass, prototype pollution, and HTTP parameter pollution vulnerabilities.
Changes
Security Impact — CVE vulnerabilities fixed by this PR:
✅ 32 CVEs resolved by this upgrade, including 5 critical 🚨 CVEs
This PR will resolve the following CVEs:
data:scheme URLs that bypass memory limits, causing unbounded memory allocation and process crashes (DoS) on Node.js. The issue affects versions starting from 0.28.0 and ignoresmaxContentLength/maxBodyLengthprotections.This issue affects form-data: < 2.5.4, 3.0.0 - 3.0.3, 4.0.0 - 4.0.3.
Breaking Changes & Upgrade Impact — prefer Warnings / Cautions /⚠️ sections over implementation sections (upgrade risks, breaking changes, manual migration steps):
⚠️ Breaking changes analysis not available for: axios
✅ No breaking changes for: form-data
Fix Details / Technical Implementation (detailed write-up of what was changed and why):
🤖 Remediation details
Fix transitive security vulnerabilities in
axiosandform-data(examples/example-node)Short summary
This PR remediates security vulnerabilities in two transitive dependencies —
axiosandform-data— within theexamples/example-nodeproject. Both packages are pulled in transitively throughlangfuse-node(andform-datafurther throughaxios). Nopackage.jsonmanifest changes were required; all fixes are lockfile-only refreshes inexamples/example-node/yarn.lock, advancing the resolved versions to patched releases.axios
axiosis a transitive dependency resolved vialangfuse-node → axios@^1.7.4. The installed version1.7.8was vulnerable across a wide range of CVEs requiring patched versions up to1.18.0. Because the parent's declared range (^1.7.4) already permitted any1.xrelease, no manifest edit was needed — a lockfile refresh viayarn upgrade 'axios@^1.7.4'was sufficient to advance the resolved version to1.20.0, satisfying all patched-version floors across the full CVE set.form-data
form-datais a transitive dependency resolved vialangfuse-node → axios → form-data@^4.0.0. The installed version4.0.0was vulnerable to two CVEs requiring≥4.0.4and≥4.0.6respectively. The parentaxios's declared range (^4.0.0) already admitted4.0.6, so a lockfile refresh viayarn upgrade 'form-data@^4.0.0'resolved it to4.0.6without any manifest change.Version changes
axios1.7.81.20.0langfuse-nodeform-data4.0.04.0.6axiosRelease info Sub-libraries affected
Bump level
Libraries affected
Changelog notes
Note
Low Risk
Scope is limited to the example app’s lockfile; it reduces known HTTP-client CVE exposure with no application source edits, though axios 1.7→1.20 may still change runtime behavior in the sample.
Overview
Refreshes
examples/example-node/yarn.lockonly (nopackage.jsonchanges) to pull patched transitive HTTP dependencies used bylangfuse-node.axiosmoves from 1.7.8 to 1.20.0, addressing a large set of reported issues (proxy/NO_PROXYbypass, prototype pollution, redirect credential leaks, size-limit bypasses, and related DoS vectors). The lockfile also picks up newerform-data(4.0.0 → 4.0.6),follow-redirects,proxy-from-env, andhttps-proxy-agent/agent-baseas required by the newer axios tree.Reviewed by Cursor Bugbot for commit 48d90f9. Bugbot is set up for automated code reviews on this repo. Configure here.