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

Adding compatibility with Pkcs5S2 certificates with HMAC256 #109

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jperlope
Copy link

@jperlope jperlope commented Feb 6, 2018

Do not use the default HMAC with default SHA1 digest anymore. We use digest indicated in pbeParameters.Prf.Algorithm. Tested with HMAC/SHA1 and HMAC/SHA256 on Pkcs5S2 certificates.

Algorithms SHA-1 and SHA-256 added.
Not use the default HMAC with default SHA1 digest anymore. We use digest indicated in pbeParameters.Prf.Algorithm. Tested with HMAC/SHA1 and HMAC/SHA256 on Pkcs5S2 certificates.
@jperlope
Copy link
Author

I want to write unit tests to verify this fix but I'm only have certificates Pkcs5S2 with HMAC256 from my organization. And I can't publish them. Could anybody tell me how to create a certificate Pkcs5S2 with HMAC256 using BC libraries, please ?

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.

None yet

1 participant