Skip to content

fix(deps): patch 3 npm vulnerabilities (brace-expansion, path-to-regexp, picomatch)#43

Merged
homeles merged 1 commit intomainfrom
security/auto-fix-2026-03-30
Mar 30, 2026
Merged

fix(deps): patch 3 npm vulnerabilities (brace-expansion, path-to-regexp, picomatch)#43
homeles merged 1 commit intomainfrom
security/auto-fix-2026-03-30

Conversation

@lupita-hom
Copy link
Copy Markdown
Collaborator

Automated Security Scan — 2026-03-30

npm audit findings (server/)

Package Severity Advisory
brace-expansion 4.0.0–5.0.4 Moderate Zero-step sequence causes process hang and memory exhaustion (GHSA-f886-m6hf-6m8v)
path-to-regexp <0.1.13 High ReDoS via multiple route parameters (GHSA-37ch-88jc-xwx2)
picomatch ≤2.3.1 High Method injection in POSIX character classes + ReDoS via extglob quantifiers (GHSA-3v7f-55p6-f55p, GHSA-c2c7-rcm5-vvqj)

Code review

Reviewed: server.js, socket.js, github.js, githubAuth.js, db.js, docker-compose.yml, .env.example, Helm chart templates, CI workflow, workflowController.js, syncService.js, workflowService.js, apiService.js (client).

No new code-level security issues found. Existing hardening (CORS allowlists, rate limiting, webhook signature verification, admin token auth, input sanitization, NoSQL injection guards) remains solid.

Changes

  • server/package-lock.json — updated via npm audit fix (no breaking changes)

Generated by Lupita's weekly security scan cron job

…xp, picomatch)

- brace-expansion 4.0.0-5.0.4: zero-step sequence DoS (GHSA-f886-m6hf-6m8v, moderate)
- path-to-regexp <0.1.13: ReDoS via multiple route params (GHSA-37ch-88jc-xwx2, high)
- picomatch <=2.3.1: method injection + ReDoS (GHSA-3v7f-55p6-f55p, GHSA-c2c7-rcm5-vvqj, high)

Applied via npm audit fix — no breaking changes.
@github-actions
Copy link
Copy Markdown

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 981a50d.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

OpenSSF Scorecard

PackageVersionScoreDetails
npm/brace-expansion 5.0.5 🟢 6.3
Details
CheckScoreReason
Packaging⚠️ -1packaging workflow not detected
Maintained🟢 1015 commit(s) and 1 issue activity found in the last 90 days -- score normalized to 10
Security-Policy🟢 10security policy file detected
Binary-Artifacts🟢 10no binaries found in the repo
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review⚠️ 2Found 7/25 approved changesets -- score normalized to 2
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Pinned-Dependencies⚠️ 0dependency not pinned by hash detected -- score normalized to 0
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 9license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ -1internal error: error during branchesHandler.setup: internal error: some github tokens can't read classic branch protection rules: https://github.com/ossf/scorecard-action/blob/main/docs/authentication/fine-grained-auth-token.md
SAST⚠️ 0SAST tool is not run on all commits -- score normalized to 0
npm/path-to-regexp 0.1.13 🟢 7.3
Details
CheckScoreReason
Binary-Artifacts🟢 10no binaries found in the repo
Branch-Protection⚠️ 1branch protection is not maximal on development and all release branches
CI-Tests🟢 821 out of 24 merged PRs checked by a CI test -- score normalized to 8
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Code-Review🟢 7found 8 unreviewed changesets out of 29 -- score normalized to 7
Contributors🟢 1025 different organizations found -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Dependency-Update-Tool🟢 10update tool detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Maintained🟢 55 commit(s) out of 30 and 1 issue activity out of 30 found in the last 90 days -- score normalized to 5
Packaging⚠️ -1no published package detected
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
SAST🟢 9SAST tool detected but not run on all commits
Security-Policy🟢 9security policy file detected
Signed-Releases⚠️ -1no releases found
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Vulnerabilities🟢 10no vulnerabilities detected
npm/picomatch 2.3.2 🟢 6.2
Details
CheckScoreReason
Packaging⚠️ -1packaging workflow not detected
Token-Permissions🟢 10GitHub workflow tokens follow principle of least privilege
Maintained🟢 1020 commit(s) and 1 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Code-Review🟢 3Found 6/20 approved changesets -- score normalized to 3
Pinned-Dependencies🟢 3dependency not pinned by hash detected -- score normalized to 3
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Fuzzing⚠️ 0project is not fuzzed
License🟢 10license file detected
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 0branch protection not enabled on development/release branches
Security-Policy🟢 10security policy file detected
SAST⚠️ 2SAST tool is not run on all commits -- score normalized to 2

Scanned Files

  • server/package-lock.json

@homeles homeles merged commit 5685f05 into main Mar 30, 2026
12 checks passed
@homeles homeles deleted the security/auto-fix-2026-03-30 branch March 30, 2026 17:51
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