Skip to content

fix: bump 8 deps and add 2 overrides to resolve Snyk vulnerabilities - #69

Open
cs-raj wants to merge 1 commit into
masterfrom
fix/snyk-patch-deps-20260810
Open

fix: bump 8 deps and add 2 overrides to resolve Snyk vulnerabilities#69
cs-raj wants to merge 1 commit into
masterfrom
fix/snyk-patch-deps-20260810

Conversation

@cs-raj

Copy link
Copy Markdown
Contributor

Summary

  • Bumped 8 direct dependencies (patch + minor, no breaking changes)
  • Added 2 overrides to pin vulnerable transitive deps
  • Bumped package version 1.1.4 → 1.1.5

Dependency Changes

PackageSectionBeforeAfter
debugdependencies^4.3.5^4.4.3
@types/basic-authdevDependencies^1.1.2^1.1.8
@types/nodedevDependencies^10.12.2^10.17.60
jestdevDependencies^29.0.3^29.7.0
rimrafdevDependencies^2.6.2^2.7.1
supertestdevDependencies^3.3.0^3.4.2
typescriptdevDependencies^4.2.0^4.9.5
winstondevDependencies^3.1.0^3.19.0

Overrides Added (transitive HIGH vulns)

PackageVulnerable RangePinned ToCVE / Issue
brace-expansion<=1.1.171.1.18DoS via unbounded expansion (OOM crash)
js-yaml3.0.0 - 3.15.03.15.1Quadratic CPU consumption — CVE-2026-59870

Before vs After

SeverityBeforeAfterFixed
HIGH20✓ 2
Moderate00
Low00

npm audit: clean ✅ Snyk SCA: clean ✅

Remaining Snyk Code findings (not fixed by dep updates — static analysis)

SeverityFinding
MEDIUMhttp.createServer cleartext transmission (×2) — intentional, tracked in DX-9946, to be suppressed via .snyk ignore
HIGHHardcoded secret in migration file
LOWHardcoded secrets in test files (×2)

Build

No build script present — library package, build verification not applicable.

🤖 Generated with Claude Code

Direct dep upgrades:
- debug: ^4.3.5 → ^4.4.3
- @types/basic-auth: ^1.1.2 → ^1.1.8
- @types/node: ^10.12.2 → ^10.17.60
- jest: ^29.0.3 → ^29.7.0
- rimraf: ^2.6.2 → ^2.7.1
- supertest: ^3.3.0 → ^3.4.2
- typescript: ^4.2.0 → ^4.9.5
- winston: ^3.1.0 → ^3.19.0
Overrides added (transitive HIGH vulns):
- brace-expansion: pinned to 1.1.18 (DoS via unbounded expansion)
- js-yaml: pinned to 3.15.1 (quadratic CPU CVE-2026-59870)
Package version: 1.1.4 → 1.1.5
Vuln count: 2 HIGH (before) → 0 (after). No build script (library package).
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@cs-raj
cs-raj requested a review from a team as a code ownerAugust 10, 2026 06:31
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

@cs-rajcs-raj closed this Aug 10, 2026
@cs-rajcs-raj reopened this Aug 10, 2026
@snyk-io

snyk-ioBot commented Aug 10, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

StatusScan Engine Critical High Medium LowTotal (0)
Open Source Security0000 0 issues
Licenses0000 0 issues
Code Security0000 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

1 similar comment
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

Sign up for freeto 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.

4 participants

@cs-raj@reeshika-h@harshitha-cstk@abhishek-ezhava-cstk