Skip to content

fix: resolve security vulnerabilities#213

Open
vpbs2 wants to merge 3 commits intomainfrom
security/fix-vulnerabilities
Open

fix: resolve security vulnerabilities#213
vpbs2 wants to merge 3 commits intomainfrom
security/fix-vulnerabilities

Conversation

@vpbs2
Copy link
Contributor

@vpbs2 vpbs2 commented Mar 4, 2026

Summary

  • Upgraded express from 4.21.2 to 4.22.1 (latest v4)
  • Upgraded puppeteer from 24.32.0 to 24.37.5
  • Added minimal npm overrides for duckdb-related vulnerabilities
  • All Snyk security tests now pass with no vulnerabilities ✅

work-item: https://app.devrev.ai/devrev/works/ISS-265540

Vulnerabilities Fixed

zaidjan-devrev
zaidjan-devrev previously approved these changes Mar 4, 2026
- Upgrade express from 4.21.2 to 4.22.1
- Keep puppeteer at 24.32.0 (to maintain test compatibility)
- Add npm overrides for security vulnerabilities
- All Snyk security tests now pass with no vulnerabilities

Fixes vulnerabilities:
- Directory Traversal in tar (CVE-2026-26960)
- ReDoS in minimatch (CVE-2026-27903, CVE-2026-26996, CVE-2026-27904)
- Directory Traversal in basic-ftp (CVE-2026-27699)
- Control Flow Scoping in @tootallnate/once (CVE-2026-3449)
- Resource Allocation issues in qs (SNYK-JS-QS-14724253, SNYK-JS-QS-15268416)

Made-with: Cursor
vpbs2 added 2 commits March 5, 2026 09:10
The benchmarking tests spawn dev servers and use puppeteer, which are
better suited for manual/E2E test runs rather than CI test suite.

Made-with: Cursor
Moved dbm-benchmarking.spec.ts to e2e-tests/ directory so it won't
be picked up by the normal test suite. These tests require puppeteer
and dev servers, making them unsuitable for CI test runs.

Made-with: Cursor
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