-
Notifications
You must be signed in to change notification settings - Fork 836
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
Initial ASCON hash256 and AEAD128 support based on NIST SP 800-232 ipd #8307
base: master
Are you sure you want to change the base?
Conversation
bd2fa1e
to
e8b9bc4
Compare
retest this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Implemented based on the NIST Initial Public Draft "NIST SP 800-232 ipd". Testing based on KAT's available at https://github.com/ascon/ascon-c. Added configuration for testing in github action.
@@ -6044,6 +6044,17 @@ then | |||
AM_CFLAGS="$AM_CFLAGS -DHAVE_XCHACHA" | |||
fi | |||
|
|||
# ASCON |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should it be part of experimental or all yet?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its a complete but limited implementation. Its fair to list it under experimental for now. Should I require WOLFSSL_EXPERIMENTAL_SETTINGS
to enable ASCON for now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes let's put ASCON under experimental for now. Thank you
Implemented based on the NIST Initial Public Draft "NIST SP 800-232 ipd". Testing based on KAT's available at https://github.com/ascon/ascon-c. Added configuration for testing in github action.