Skip to content

deps(security): axios 1.10.0 -> 1.16.1 - #342

Open
luantruong6 wants to merge 1 commit into
masterfrom
deps/security-axios-1.16.1
Open

deps(security): axios 1.10.0 -> 1.16.1#342
luantruong6 wants to merge 1 commit into
masterfrom
deps/security-axios-1.16.1

Conversation

@luantruong6

@luantruong6 luantruong6 commented May 29, 2026

Copy link
Copy Markdown

Security issue addressed

Tracking ticket

  • KVDP/Jira: KVDP-9005
  • Ticket status: existing
  • Owner/queue: Unassigned

What changed

  • Updated package.json direct dependency axios from ^1.6.8 to ^1.16.1.
  • Updated package-lock.json to resolve axios from 1.10.0 to 1.16.1.
  • Refreshed axios transitive lockfile entries, including follow-redirects, form-data, https-proxy-agent, and proxy-from-env resolution changes required by axios@1.16.1.

Expected impact

  • Security impact: Removes the vulnerable locked axios@1.10.0 version identified by Aikido and resolves to the patched 1.16.1 target.
  • Functional impact: No user-facing behavior expected; axios remains on the same major version.
  • Operational impact: None expected.

Verification evidence

  • Tests run: mise exec node@20.9.0 -- npm ci — pass; install completed from the refreshed lockfile.
  • Tests run: mise exec node@20.9.0 -- npm run eslint — pass with existing warnings only (0 errors, 10 warnings).
  • Tests run: mise exec node@20.9.0 -- npm run test:coverage — fail; existing Jest failures in test/service/command-executor.test.js plus timeout. The CI workflow currently runs this command with || true.
  • Tests run: mise exec node@20.9.0 -- npm run build — pass; esbuild produced agent-bundle.js successfully.
  • Tests run: mise exec node@20.9.0 -- npm ls axios --depth=0 — pass; reports axios@1.16.1.
  • Security validation: mise exec node@20.9.0 -- npm audit --omit=dev --json — no axios entry in remaining production audit vulnerabilities; unrelated vulnerabilities remain in other packages.
  • Compatibility check: CI and local runtime inspected as Node.js 20.9.0 via .nvmrc, GitHub Actions matrices, and Dockerfile; local verification used mise exec node@20.9.0.

Rollout and compatibility risk

  • Risk level: low
  • Compatibility notes: Same-major axios upgrade. Lockfile changes include axios transitive dependency updates expected for the patched release.
  • Rollback plan: Revert this PR.

Ownership and review notes

  • Suggested reviewers/owners: Repo owners
  • Follow-ups: KVDP-9005 tracks this remediation. Remaining non-axios dependency findings should be handled separately.
  • Human decision required: Engineering review/approval is required before merge.

Summary by Aikido

Security Issues: 0 Quality Issues: 0 ✅ Resolved Issues: 19

⚡ Enhancements

  • Updated axios dependency from ^1.6.8 to ^1.16.1 in package.json

More info

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.

1 participant