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

Remove BouncyCastle #5

Open
MariusVolkhart opened this issue Feb 9, 2020 · 0 comments
Open

Remove BouncyCastle #5

MariusVolkhart opened this issue Feb 9, 2020 · 0 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@MariusVolkhart
Copy link
Owner

The syncthing-core module declares a dependency on BouncyCastle Mail that we want removed.

It's used for 3 things: Hex encoding, Base64 encoding, and converting the initial client keypair for storage in a Keystore. We have our own Hex already, Base64 is built into Java now, and the key logic should be achievable using only JCA classes. Yes, that would require a Android minSdk of 26, but that's now our main target.

@MariusVolkhart MariusVolkhart added enhancement New feature or request good first issue Good for newcomers labels Feb 9, 2020
@MariusVolkhart MariusVolkhart self-assigned this May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant