-
Notifications
You must be signed in to change notification settings - Fork 1
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 support for AES-CCM #24
Conversation
This PR should now be ready for review. Aside from the added features (AES-CCM and partial IVs), I have also applied some (new?) clippy suggestions for old code in 4b33e00. |
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.
A record low of only 6 review comments from me! And these comments are only very minor, so I expect we can merge this very soon.
Co-authored-by: Falko Galperin <[email protected]>
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.
Looks good to me now 👍🏻
This PR adds support for all AES-CCM variants to the COSE module and both cryptographic backends.
Part of #18.
Additionally, this PR makes some smaller optimizations to the AES-GCM code (saving one copy operation and making it consistent with the AES-CCM code) and adds support for partial IVs.