Skip to content

SCANJLIB-324 Relocate bouncycastle to fix the conflict with versions already present on Gradle / AGP - #302

Open
alban-auzeill wants to merge 1 commit into
masterfrom
alban/relocate-bouncycastle
Open

SCANJLIB-324 Relocate bouncycastle to fix the conflict with versions already present on Gradle / AGP#302
alban-auzeill wants to merge 1 commit into
masterfrom
alban/relocate-bouncycastle

Conversation

@alban-auzeill

@alban-auzeill alban-auzeill commented Aug 20, 2026

Copy link
Copy Markdown
Member

Summary by Gitar

  • Dependency Relocation:
    • Created bouncycastle-shaded module using maven-shade-plugin to repackage bcprov-jdk18on under org.sonarsource.scanner.lib.internal.impldep
    • Updated lib module dependencies and imports to use the shaded BouncyCastle artifact and package paths

This will update automatically on new commits.

@alban-auzeill
alban-auzeill requested a review from a team as a code owner August 20, 2026 14:26
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod Bot changed the title Relocate bouncycastle to fix the conflict with versions already present on Gradle / AGP SCANJLIB-324 Relocate bouncycastle to fix the conflict with versions already present on Gradle / AGP Aug 20, 2026
@hashicorp-vault-sonar-prod

hashicorp-vault-sonar-prod Bot commented Aug 20, 2026

Copy link
Copy Markdown

SCANJLIB-324

@alban-auzeill
alban-auzeill force-pushed the alban/relocate-bouncycastle branch from a61ef6d to e9ef136 Compare August 20, 2026 14:27
@gitar-bot

gitar-bot Bot commented Aug 20, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Relocates bouncycastle into a shaded module to resolve classpath conflicts with Gradle and AGP. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@sonarqube-next

Copy link
Copy Markdown

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