Skip to content

fix(frontend): upgrade deps to resolve HIGH/MODERATE CVEs (axios, react-router-dom) - #48

Merged
moonandlife merged 1 commit into
mainfrom
fix/frontend-security-deps
Apr 3, 2026
Merged

fix(frontend): upgrade deps to resolve HIGH/MODERATE CVEs (axios, react-router-dom)#48
moonandlife merged 1 commit into
mainfrom
fix/frontend-security-deps

Conversation

@moonandlife

Copy link
Copy Markdown
Contributor

Summary

Upgrades three frontend dependencies to patch production-runtime security vulnerabilities identified by npm audit.

Changes

Package Before After Advisory
axios ^1.6.2 ^1.14.0 GHSA-43fc-jf86-j433 DoS via __proto__ in mergeConfig — HIGH CVSS 7.5
react-router-dom ^6.20.1 ^6.30.3 GHSA-2w69-qvjg-hvjx XSS via open redirect — HIGH CVSS 8.0
react-syntax-highlighter ^16.1.0 ^16.1.1 patch update

Transitive packages also resolved: @remix-run/router 1.23.1 → 1.23.2, react-router 6.30.2 → 6.30.3.

Impact

  • Vulnerability count: 19 → 15 (HIGH: 13 → 9)
  • All resolved findings were production-runtime packages shipped in the browser bundle
  • Remaining 15 advisories are dev-toolchain only (eslint, vite, vitest, rollup) — addressed separately in the Priority 3/4 ESLint 9 + Vite 6 migration

Testing

No API surface changes — these are patch/minor upgrades. Existing tests and build should be unaffected.

- axios ^1.6.2 -> ^1.14.0: fixes GHSA-43fc-jf86-j433 (DoS via __proto__ in mergeConfig, HIGH CVSS 7.5)
- react-router-dom ^6.20.1 -> ^6.30.3: fixes GHSA-2w69-qvjg-hvjx (XSS via open redirect, HIGH CVSS 8.0)
- react-syntax-highlighter ^16.1.0 -> ^16.1.1: patch update

Transitive fixes: @remix-run/router 1.23.1 -> 1.23.2, react-router 6.30.2 -> 6.30.3
Vulnerability count: 19 -> 15 (13 HIGH -> 9 HIGH); remaining are dev-toolchain only
@moonandlife
moonandlife merged commit 9694a62 into main Apr 3, 2026
3 checks passed
@moonandlife
moonandlife deleted the fix/frontend-security-deps branch April 3, 2026 09:02
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