Skip to content

fix(security): resolve all npm audit vulnerabilities (2026-08-24) - #30

Merged
homeles merged 1 commit into
mainfrom
security/auto-fix-2026-08-24
Aug 24, 2026
Merged

fix(security): resolve all npm audit vulnerabilities (2026-08-24)#30
homeles merged 1 commit into
mainfrom
security/auto-fix-2026-08-24

Conversation

@lupita-hom

Copy link
Copy Markdown
Collaborator

Security Scan — 2026-08-24

Client (5 vulnerabilities fixed)

Package Severity Issue
@babel/core low Arbitrary file read via sourceMappingURL
nanoid high Infinite loop with negative/zero size
postcss high XSS, path traversal, arbitrary file read
react-router high RCE, DoS, CSRF, open redirect, XSS
vite high NTLMv2 hash disclosure, fs.deny bypass

Server (4 vulnerabilities fixed)

Package Severity Issue
body-parser moderate DoS via invalid limit value
esbuild low Arbitrary file read on Windows dev server
qs moderate DoS via TypeError on null entries
express moderate Depends on vulnerable qs

Code Review

Manual review of server routes, auth, CORS, services, and Docker config found no additional issues:

  • ✅ OAuth state validation present
  • ✅ Auth middleware on all API routes
  • ✅ CORS restricted to configured origin
  • ✅ Helmet security headers enabled
  • ✅ No secrets in code (env-based config)
  • ✅ No injection vectors in route params
  • ✅ Session cookies: httpOnly + sameSite=lax

All fixes are non-breaking semver updates via npm audit fix.

Client (5 fixed):
- @babel/core: arbitrary file read via sourceMappingURL
- nanoid: infinite loop with negative/zero size (high)
- postcss: XSS, path traversal, arbitrary file read (high)
- react-router: RCE, DoS, CSRF, open redirect, XSS (high)
- vite: NTLMv2 hash disclosure, fs.deny bypass on Windows (high)

Server (4 fixed):
- body-parser: DoS via invalid limit value (moderate)
- esbuild: arbitrary file read on Windows dev server
- qs: DoS via TypeError on null/undefined entries (moderate)
- express: depends on vulnerable qs (moderate)

All resolved via npm audit fix (non-breaking semver updates).
@homeles
homeles merged commit 5790205 into main Aug 24, 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