-
-
Notifications
You must be signed in to change notification settings - Fork 81
feat(registry): add AES-CMAC-PRF-128 (RFC 4615) #768
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
feat(registry): add AES-CMAC-PRF-128 (RFC 4615) #768
Conversation
Signed-off-by: Mehrn0ush <[email protected]>
bhess
left a comment
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.
LGTM.
One change request: remove the "HPKE" avoid merge conflicts with #766
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.
Pull request overview
This PR adds support for two new cryptographic algorithms to the registry: HPKE (Hybrid Public Key Encryption, RFC 9180) and AES-CMAC-PRF-128 (RFC 4615). However, the PR title and description only mention AES-CMAC-PRF-128, omitting the HPKE addition.
Changes:
- Added HPKE algorithm family with RFC 9180 standard reference
- Added AES-CMAC-PRF-128 variant under the existing AES family with RFC 4615 standard reference
Comments suppressed due to low confidence (2)
schema/cryptography-defs.json:11
- Trailing whitespace after the closing brace and comma. This is inconsistent with the formatting used elsewhere in the file where closing braces have no trailing spaces.
},
schema/cryptography-defs.json:11
- Trailing whitespace after the closing brace and comma. This is inconsistent with the formatting used elsewhere in the file where closing braces have no trailing spaces.
},
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Mehrn0ush <[email protected]>
|
Removed HPKE entry to avoid conflicts and fixed RFC formatting for AES-CMAC-PRF-128. |
As discussed in ticket #767, this PR adds AES-CMAC-PRF-128 (RFC 4615) to the Cryptography Registry.
Fixes #767
Details
AES-CMAC-PRF-128as amacvariant under the existingAESfamily.schema/cryptography-defs.json). No schema or specification behavior changes.