Skip to content

Allow LGPL-3.0 for sonarqube-scan-action in dependency review - #286

Merged
AndreasIgel merged 1 commit into
java-helpers:mainfrom
AndreasIgel:devin/dep-review-sonar-scan-license
Sep 12, 2026
Merged

AndreasIgel merged 1 commit into
java-helpers:mainfrom
AndreasIgel:devin/dep-review-sonar-scan-license

Conversation

@AndreasIgel

Copy link
Copy Markdown
Collaborator

Summary

  • Adds a scoped allow-dependencies-licenses exception for pkg:githubactions/SonarSource/sonarqube-scan-action, whose LGPL-3.0 license is outside the fail-closed allow-licenses allowlist.
  • The action only executes the scanner on the runner — use of an unmodified tool, not distribution or a derivative work — so LGPL obligations do not apply.
  • The exception purl is deliberately version-less: dependency-review-action's matcher ignores the version component, and it is the usage as a CI action that makes the license unproblematic.
  • The strict allow-licenses policy is unchanged for all other dependencies (including Maven artifacts, where LGPL genuinely matters).

Context

This unblocks the dependency-review check on #285, which introduces SonarSource/sonarqube-scan-action in fork-sonar.yml. Since this workflow runs on pull_request_target, the exception is read from the base branch — it must merge to main before #285's check can go green (re-run the failed job after merging).

Test plan

The action only executes the scanner on the runner — use of an
unmodified tool, not distribution — so LGPL obligations do not apply.
Scope the exception via allow-dependencies-licenses so the fail-closed
allow-licenses list stays strict for all other dependencies.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@AndreasIgel
AndreasIgel merged commit 44d00ed into java-helpers:main Sep 12, 2026
5 checks passed
@AndreasIgel
AndreasIgel deleted the devin/dep-review-sonar-scan-license branch September 12, 2026 16:17
@codecov

codecov Bot commented Sep 12, 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!

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