Skip to content

Fix npm audit vulnerabilities and pin axios to safe version#142

Merged
OndrejKotek merged 2 commits intohal:mainfrom
kstekovi:EAPQE-5257
Apr 8, 2026
Merged

Fix npm audit vulnerabilities and pin axios to safe version#142
OndrejKotek merged 2 commits intohal:mainfrom
kstekovi:EAPQE-5257

Conversation

@kstekovi
Copy link
Copy Markdown
Collaborator

@kstekovi kstekovi commented Apr 7, 2026

Summary

  • Pin axios to exact version 1.13.6 to prevent automatic installation of potentially compromised versions via semver range resolution
  • Add npm overrides for mocha transitive dependencies (diff and serialize-javascript) to resolve all remaining npm audit vulnerabilities
  • diff overridden to ^8.0.3 (fixes DoS in parsePatch/applyPatch)
  • serialize-javascript overridden to ^7.0.5 (fixes RCE and CPU exhaustion DoS)
  • Result: npm audit reports 0 vulnerabilities

Test plan

  • Run npm install and verify it completes successfully
  • Run npm audit and verify 0 vulnerabilities
  • Run existing test suite to ensure no regressions from dependency updates

🤖 Generated with Claude Code

Add overrides for diff (^8.0.3) and serialize-javascript (^7.0.5) to
resolve all remaining npm audit vulnerabilities in mocha's dependency tree.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@kstekovi kstekovi changed the title EAPQE-5257 Fix npm audit vulnerabilities Fix npm audit vulnerabilities and pin axios to safe version Apr 8, 2026
Pin axios to exact version 1.13.6 to prevent automatic installation of
potentially compromised versions via semver range resolution.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
@OndrejKotek OndrejKotek merged commit 1f07134 into hal:main Apr 8, 2026
1 check passed
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.

2 participants