Skip to content
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

Switch the BC dependency over to the official package #160

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

Daniel-King-GGL
Copy link
Contributor

BouncyCastle has their own official nuget package - any reason we shouldn't switch over?
https://www.nuget.org/packages/BouncyCastle.Cryptography

@tintoy
Copy link
Owner

tintoy commented Aug 6, 2024

Hi - thanks for contributing! I suspect that dependency was added before there was an official package but it would be much nicer to use the office one.

We don’t currently have much in the way of test coverage for our usage of BouncyCastle so I’ll need to add a test or 2 just to make sure the core use-cases don’t break 🙂

(hopefully sometime tonight)

@tintoy tintoy self-assigned this Aug 6, 2024
@tintoy tintoy added the enhancement New feature or request label Aug 6, 2024
tintoy added a commit that referenced this pull request Aug 7, 2024
@tintoy tintoy merged commit c4ce8dc into tintoy:develop Aug 7, 2024
1 check passed
tintoy added a commit that referenced this pull request Aug 7, 2024
@tintoy
Copy link
Owner

tintoy commented Aug 7, 2024

Thanks - it's merged and the tests pass :)

tintoy added a commit that referenced this pull request Aug 7, 2024
@tintoy
Copy link
Owner

tintoy commented Aug 7, 2024

Published v2.5.9-develop.11 to MyGet. Can you give it a try and let me know if it works for you?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants