Standard.Licensing icon indicating copy to clipboard operation
Standard.Licensing copied to clipboard

Update BouncyCastle to latest version

Open ahmadreza-hadidi opened this issue 1 year ago • 1 comments

Hi, @junian Thank you for your good work Please update BouncyCastle to latest version because of vulnerability in older versions

ahmadreza-hadidi avatar Jun 29 '24 06:06 ahmadreza-hadidi

Hi @ahmadreza-hadidi, Can you give reference which vulnerability?

junian avatar Jul 01 '24 01:07 junian

https://github.com/advisories/GHSA-8xfc-gm6g-vgpv https://github.com/advisories/GHSA-v435-xc8x-wvr9 https://github.com/advisories/GHSA-8xfc-gm6g-vgpv

image

You're also using a forked version, it looks like. Can you please use the standard version? https://www.nuget.org/packages/BouncyCastle.Cryptography

They updated it to remove the need to use the portable version when they depreciated the BouncyCastle.Crypto version.

kfrancis avatar Jul 08 '24 17:07 kfrancis

https://www.nuget.org/packages/BouncyCastle

kfrancis avatar Jul 08 '24 17:07 kfrancis

I do understand that you're using 1.9.0, but there's an official package so you don't need to reference the external repo.

kfrancis avatar Jul 08 '24 17:07 kfrancis

This is the other reason why it's important. Can't use packages with vulnerabilities, and can't use the standard library because yours brings in a transitive reference:

image

I've put a pull request together to fix all that up.

kfrancis avatar Jul 08 '24 17:07 kfrancis

@junian Have you had a chance to check the pull request?

kfrancis avatar Jul 19 '24 00:07 kfrancis

Thank you. I'll review it soon.

junian avatar Jul 23 '24 05:07 junian

Appreciate it. It's somewhat urgent.

kfrancis avatar Jul 29 '24 16:07 kfrancis

Thank you @kfrancis for the contribution, it's being validated on NuGet right now, should be available soon.

junian avatar Jul 31 '24 01:07 junian

The release works great, thank you. Also, I can confirm that the change in BouncyCastle doesn't require a change in existing licenses - they work as is.

kfrancis avatar Aug 01 '24 15:08 kfrancis