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 TurboSHAKE #458

Merged
merged 2 commits into from
Mar 17, 2023
Merged

Add TurboSHAKE #458

merged 2 commits into from
Mar 17, 2023

Conversation

aewag
Copy link
Contributor

@aewag aewag commented Mar 15, 2023

This adds TurboSHAKE128 and TurboSHAKE256 to the sha3 crate.
It is based on the specification document and the IETF draft.

I changed the Sha3State to be based by default on a KeccakF1600 permutation.
But the round_count can be overwritten using new() to base the Sha3State on KeccakP1600.
To reflect this change, I renamed the function apply_f to permute.

This implementation is based on the specification at
 https://eprint.iacr.org/2023/342
and the IETF draft at
 https://www.ietf.org/id/draft-irtf-cfrg-kangarootwelve-09.html
@tarcieri
Copy link
Member

@aewag nice! Do you plan to use this in the k12 PR?

@aewag
Copy link
Contributor Author

aewag commented Mar 15, 2023

@aewag nice! Do you plan to use this in the k12 PR?

Yes, that's planned, and I hope I'll find the time to finish the k12PR soon.

@aewag aewag force-pushed the add-turbo-shake branch 2 times, most recently from 0c7db92 to e7e3b44 Compare March 16, 2023 10:01
@aewag aewag marked this pull request as ready for review March 16, 2023 12:23
@aewag
Copy link
Contributor Author

aewag commented Mar 16, 2023

@tarcieri I added the test vectors as .bbl files and updated the tests. With this the PR is ready for review. :)

@tarcieri tarcieri requested a review from newpavlov March 16, 2023 15:56
@tarcieri tarcieri merged commit 9b218cf into RustCrypto:master Mar 17, 2023
@aewag aewag mentioned this pull request May 7, 2023
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.

2 participants