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

add hmac_sha384 function #8

Merged
merged 1 commit into from
Apr 24, 2022
Merged

Conversation

dpatti
Copy link
Contributor

@dpatti dpatti commented May 4, 2018

Using the existing HMAC functor, apply the Hash.sha384 class to
create a module and exposed function for creating HMAC hashes with the
SHA-384 algorithm.

The inputs for these tests were copied from the HMAC-SHA512 tests below
it. For each test, I made sure that I could produce the expected
HMAC-SHA512 output using openssl sha512 -hmac on the command line, and
then I switched sha512 to sha384. The resulting output was used as
the expected value for each of the tests, and they all passed. This
makes me reasonably confident in its correctness.

Using the existing `HMAC` functor, apply the `Hash.sha384` class to
create a module and exposed function for creating HMAC hashes with the
SHA-384 algorithm.

The inputs for these tests were copied from the HMAC-SHA512 tests below
it. For each test, I made sure that I could produce the expected
HMAC-SHA512 output using `openssl sha512 -hmac` on the command line, and
then I switched `sha512` to `sha384`. The resulting output was used as
the expected value for each of the tests, and they all passed. This
makes me reasonably confident in its correctness.
@wiml
Copy link

wiml commented Apr 13, 2019

This would be rather useful for me as well!

Copy link
Owner

@xavierleroy xavierleroy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I completely forgot about this PR, apologies for leaving it hanging for so long. This looks good to me. Thanks!

@xavierleroy xavierleroy merged commit dd36602 into xavierleroy:master Apr 24, 2022
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.

3 participants