Skip to content

Update org.bouncycastle:bcprov-lts8on - #1622

Merged
scottf merged 2 commits into
mainfrom
bcprov-2-73-12-1
Sep 1, 2026
Merged

Update org.bouncycastle:bcprov-lts8on#1622
scottf merged 2 commits into
mainfrom
bcprov-2-73-12-1

Conversation

@scottf

@scottf scottf commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Update to 2.73.12.1 to resolve several CVEs. See Issue #1621

Update to 2.73.12.1 to resolve several CVEs. See Issue #1621
@synadia-claude-reviewer

synadia-claude-reviewer Bot commented Sep 1, 2026

Copy link
Copy Markdown

Claude finished @scottf's task in 49s —— View job


PR Review: Update org.bouncycastle:bcprov-lts8on

  • Gather context
  • Review PR diff
  • Analyze security implications of CVEs addressed
  • Post review findings

Summary

This is a targeted security patch — a single-line version bump of org.bouncycastle:bcprov-lts8on from 2.73.102.73.12.1 in build.gradle, addressing 10 CVEs reported in Issue #1621.

Change is correct and appropriate. No issues found.

Details

Correctness: The version bump is a maintenance update within the LTS branch for Java 8+. Skipping 2.73.11 is normal (intermediate releases sometimes exist only briefly before a follow-on patch). The lts8on artifact is the correct variant for this project's Java 8 target (sourceCompatibility = JavaVersion.VERSION_1_8).

Dependency scope: bcprov-lts8on is declared as api (not implementation), so the patched version will be transitively exposed to downstream consumers — which is appropriate since BouncyCastle types appear in the public API.

CVEs addressed: All 10 CVEs listed in Issue #1621 are resolved by this version. The fix is minimal and focused, with no unrelated changes bundled in — exactly the right scope for a security patch.

No concerns with thread safety, API compatibility, or build configuration.

@scottf
scottf merged commit 1c63577 into main Sep 1, 2026
5 checks passed
@scottf
scottf deleted the bcprov-2-73-12-1 branch September 1, 2026 21:16
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