chore(security): update vulnerable transitive dependencies#3245
chore(security): update vulnerable transitive dependencies#3245jrossi wants to merge 1 commit intotriggerdotdev:mainfrom
Conversation
Update transitive dependencies to address known CVEs: - semver 7.5.0 → 7.7.4 (CVE-2022-25883 ReDoS) - ws 8.11.0 → 8.19.0 (CVE-2024-37890 DoS via headers) - minimatch 10.0.1 → 10.2.4 (multiple ReDoS CVEs) - tar 7.5.4 → 7.5.11 (multiple path traversal CVEs) Packages updated: - apps/webapp: semver, ws - packages/cli-v3: minimatch, semver, tar, ws - packages/trigger-sdk: ws Test results: - @trigger.dev/core: 412/412 passed - @trigger.dev/sdk: 10/10 passed - 7/8 package test suites passed (redis-worker requires testcontainers/Docker environment, not affected by these changes)
|
|
Hi @jrossi, thanks for your interest in contributing! This project requires that pull request authors are vouched, and you are not in the list of vouched users. This PR will be closed automatically. See https://github.com/triggerdotdev/trigger.dev/blob/main/CONTRIBUTING.md for more details. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
WalkthroughThis change updates dependency versions across three package configuration files in the repository. The Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can suggest fixes for GitHub Check annotations.Configure the |
Summary
Updates transitive npm dependencies to address known security vulnerabilities (CVEs) found via Trivy image scanning.
Dependency Updates
semverwsminimatchtarPackages affected
apps/webapp: semver, wspackages/cli-v3: minimatch, semver, tar, wspackages/trigger-sdk: wsHow it was found
Running Trivy and Grype container image scans against the v4.4.3 Docker images flagged 109 CRITICAL/HIGH CVEs in the webapp image. All are in npm transitive dependencies — the OS base has zero CVEs (using Docker Hardened Images).
Test plan
@trigger.dev/core: 412/412 tests passed@trigger.dev/sdk: 10/10 tests passedredis-workerfailures are testcontainers/Docker environment issues unrelated to these changes)Notes
pnpm-lock.yamland 3package.jsonfiles changed🤖 Generated with Claude Code