Skip to content

[DO NOT MERGE] Test: dependency-review gates should fail on GPL + vulnerable deps - #219

Closed
AndreasIgel wants to merge 1 commit into
java-helpers:mainfrom
AndreasIgel:test/dependency-review-gate-firing
Closed

[DO NOT MERGE] Test: dependency-review gates should fail on GPL + vulnerable deps#219
AndreasIgel wants to merge 1 commit into
java-helpers:mainfrom
AndreasIgel:test/dependency-review-gate-firing

Conversation

@AndreasIgel

Copy link
Copy Markdown
Collaborator

⚠️ DO NOT MERGE — dependency-review gate test

This PR deliberately introduces two bad dependencies in example/pom.xml to prove the dependency-review gates actually fire. It is a draft and must not be merged.

Dependency Why it should fail
com.mysql:mysql-connector-j:9.1.0 GPL-2.0 licensed → trips the license gate (deny-licenses on main; allow-licenses once #218 merges).
org.apache.logging.log4j:log4j-core:2.14.1 Log4Shell (CVE-2021-44228, critical) → trips the severity gate (fail-on-severity: high).

Expected result

The dependency-review check turns this PR red, reporting the denied license and the critical vulnerability. That is the pass condition for this experiment.

After you've confirmed the check goes red, this PR should be closed (not merged). Fork PRs need a maintainer "Approve and run" click before the check executes.

Co-Authored-By: Andreas Igel <andreas.igel@computacenter.com>
@github-actions

github-actions Bot commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

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

Snapshot Warnings

⚠️: No snapshots were found for the head SHA deb1f9e.
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.

Vulnerabilities

example/pom.xml

NameVersionVulnerabilitySeverity
org.apache.logging.log4j:log4j-core2.14.1Incomplete fix for Apache Log4j vulnerabilitycritical
Remote code injection in Log4jcritical
Apache Log4j2 vulnerable to Improper Input Validation and Uncontrolled Recursionhigh
Only included vulnerabilities with severity high or higher.

License Issues

example/pom.xml

PackageVersionLicenseIssue Type
com.mysql:mysql-connector-j9.1.0GPL-2.0-only WITH Universal-FOSS-exception-1.0 AND LicenseRef-github-OTHERIncompatible License
Allowed Licenses: MIT, Apache-2.0, BSD-2-Clause, BSD-3-Clause, EPL-1.0, EPL-2.0, CDDL-1.0, CDDL-1.1, ISC, Unlicense, CC0-1.0

OpenSSF Scorecard

PackageVersionScoreDetails
maven/org.apache.logging.log4j:log4j-core 2.14.1 UnknownUnknown
maven/com.mysql:mysql-connector-j 9.1.0 🟢 5.5
Details
CheckScoreReason
Packaging⚠️ -1packaging workflow not detected
Code-Review⚠️ 0Found 0/30 approved changesets -- score normalized to 0
Maintained🟢 1013 commit(s) and 0 issue activity found in the last 90 days -- score normalized to 10
Dangerous-Workflow⚠️ -1no workflows found
Security-Policy🟢 10security policy file detected
Token-Permissions⚠️ -1No tokens found
CII-Best-Practices⚠️ 0no effort to earn an OpenSSF best practices badge detected
SAST⚠️ 0no SAST tool detected
Binary-Artifacts🟢 10no binaries found in the repo
Pinned-Dependencies⚠️ -1no dependencies found
License🟢 9license file detected
Fuzzing🟢 10project is fuzzed
Signed-Releases⚠️ -1no releases found
Branch-Protection⚠️ 0branch protection not enabled on development/release branches

Scanned Files

  • example/pom.xml

@codecov

codecov Bot commented Jul 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@sonarqubecloud

Copy link
Copy Markdown

@AndreasIgel

Copy link
Copy Markdown
Collaborator Author

Test succeeded: dependency-review failed as designed (GPL-2.0 license + Log4Shell/CVE-2021-44228 critical vuln both tripped the gates). Closing without merging.

@devin-ai-integration
devin-ai-integration Bot deleted the test/dependency-review-gate-firing branch August 16, 2026 21:08
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