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

SHA3 2.0 specification #1065

Merged
merged 1 commit into from
Nov 17, 2020
Merged

SHA3 2.0 specification #1065

merged 1 commit into from
Nov 17, 2020

Conversation

Kritner
Copy link
Contributor

@Kritner Kritner commented Nov 17, 2020

@Kritner Kritner merged commit 340e547 into usnistgov:master Nov 17, 2020
@smuellerDD
Copy link
Contributor

smuellerDD commented Nov 17, 2020 via email

@Kritner
Copy link
Contributor Author

Kritner commented Nov 17, 2020

ah yeah thanks

@smuellerDD
Copy link
Contributor

Just a remark: I am registering my SHA-3 already the same way as SHA-2 that includes the message length without errors.

That said, I tried it again and got no errors either. Is the version 2.0 requirement already enforced? If not, what happens if I use 1.0 and send a message length for SHA-3?

@Kritner
Copy link
Contributor Author

Kritner commented Nov 17, 2020

Properties that don't apply to a registration are simply ignored, so messageLen on a 1.0 registration just means "nothing" to the scope of testing or validation.

2.0 testing should be available on demo shortly but is not currently.

@smuellerDD
Copy link
Contributor

smuellerDD commented Nov 17, 2020 via email

@Kritner
Copy link
Contributor Author

Kritner commented Nov 17, 2020

there was no change to the sha1/2 registration nor testing, so it is still revision 1.0. The 2.0 revision of testing for sha3 was simply to have a "more sha1/2 like registration feel", without breaking the existing sha3 1.0 registration properties.

EDIT: I should also note that both the "1.0" and "2.0" revisions of sha3 testing are going to continue to be "valid". The 2.0 registration was introduced for another reason of being able to support IUTs that don't support all message lengths that the 1.0 testing would test against. (Though the digest size and 3x the digest size are still required message lengths for the IUT to support due to MCT testing)

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