Skip to content

[deps] upgrade BC to latest 1.74 #278

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jun 20, 2023
Merged

[deps] upgrade BC to latest 1.74 #278

merged 1 commit into from
Jun 20, 2023

Conversation

chadlwilson
Copy link
Contributor

@chadlwilson chadlwilson commented Jun 17, 2023

Bumps BouncyCastle to latest patched version.

Not sure if jruby-openssl uses the vulnerable LDAP stuff mentioned in CVE-2023-33201 however this is probably good to reduce noise.

It also has the positive side effect of reducing uber-jar size, as "The now defunct PQC SIKE algorithm has been removed" which seems to reduce by ~3MB.

https://www.bouncycastle.org/releasenotes.html#r1rv74

@kares
Copy link
Member

kares commented Jun 19, 2023

Thanks Chad, will take a look - there's a couple of items that I will need to find free time to work on before pushing a release.

Reading the CVE I do not see JOSSL affected by this, hopefully I understood correctly and this can be tagged out for JRuby.

@chadlwilson
Copy link
Contributor Author

@kares Yeah, I don't think relevant to JRuby users either, but you know those noisy CVE scanners. 😅 Currently the CVE hasn't been disclosed via MITRE/NIST (still RESERVED here) so it's not yet linked to bouncycastle CPE but when it is I guess folks may start complaining about transitive noise via jruby-complete and friends.

As a side note, I see there are no automated tests run against a JRuby 9.4.x release currently - not sure if that is of concern.

@kares kares merged commit 2885017 into jruby:master Jun 20, 2023
@chadlwilson chadlwilson deleted the bump-bc branch June 20, 2023 10: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.

2 participants