Skip to content

Test: dependency review must fail on a known-vulnerable dependency - #13

Closed
toxakara wants to merge 1 commit into
mainfrom
test/dependency-review-seeded
Closed

toxakara wants to merge 1 commit into
mainfrom
test/dependency-review-seeded

Conversation

@toxakara

Copy link
Copy Markdown
Member

Do not merge. Adds lodash 4.17.20 (GHSA-35jh-r3h4-6jhm, GHSA-29mw-wpgm-hmr9) to check that the dependency-review job blocks it. This PR will be closed without merging and its branch deleted.

@github-actions

Copy link
Copy Markdown

Dependency Review

The following issues were found:
  • ❌ 1 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ✅ 0 package(s) with unknown licenses.
See the Details below.

Vulnerabilities

package.json

NameVersionVulnerabilitySeverity
lodash4.17.20Command Injection in lodashhigh
lodash vulnerable to Code Injection via `_.template` imports key nameshigh
Regular Expression Denial of Service (ReDoS) in lodashmoderate
Lodash has Prototype Pollution Vulnerability in `_.unset` and `_.omit` functionsmoderate
lodash vulnerable to Prototype Pollution via array path bypass in `_.unset` and `_.omit`moderate

OpenSSF Scorecard

PackageVersionScoreDetails
npm/lodash 4.17.20 🟢 6.6
Details
CheckScoreReason
Code-Review🟢 7Found 22/30 approved changesets -- score normalized to 7
Dependency-Update-Tool🟢 10update tool detected
Maintained🟢 32 commit(s) and 2 issue activity found in the last 90 days -- score normalized to 3
Packaging⚠️ -1packaging workflow not detected
Dangerous-Workflow🟢 10no dangerous workflow patterns detected
Security-Policy🟢 10security policy file detected
Token-Permissions⚠️ 0detected GitHub workflow tokens with excessive permissions
Pinned-Dependencies🟢 4dependency not pinned by hash detected -- score normalized to 4
Binary-Artifacts🟢 10no binaries found in the repo
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
Vulnerabilities⚠️ 0120 existing vulnerabilities detected
SAST🟢 9SAST tool detected but not run on all commits
License🟢 9license file detected
Signed-Releases⚠️ -1no releases found
Fuzzing🟢 10project is fuzzed
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
CI-Tests🟢 822 out of 25 merged PRs checked by a CI test -- score normalized to 8
Contributors🟢 10project has 90 contributing companies or organizations

Scanned Files

  • package.json

@toxakara

Copy link
Copy Markdown
Member Author

Test complete: dependency review failed as intended. Closing without merging.

@toxakara toxakara closed this Oct 21, 2026
@toxakara
toxakara deleted the test/dependency-review-seeded branch October 21, 2026 04:06
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.

1 participant