Skip to content

Add support for the Ascon family of algorithms#297

Merged
athoelke merged 4 commits into
ARM-software:mainfrom
athoelke:crypto-ascon
Oct 24, 2025
Merged

Add support for the Ascon family of algorithms#297
athoelke merged 4 commits into
ARM-software:mainfrom
athoelke:crypto-ascon

Conversation

@athoelke
Copy link
Copy Markdown
Contributor

@athoelke athoelke commented Sep 12, 2025

This adds support for all four algorithms defined in NIST SP800-232.

It uses the new API merged in #253.

Fixes #60

@athoelke athoelke added this to the Crypto API 1.4 milestone Sep 12, 2025
@athoelke athoelke self-assigned this Sep 12, 2025
@athoelke athoelke added enhancement New feature or request API design Related the design of the API Crypto API Issue or PR related to the Cryptography API labels Sep 12, 2025
@athoelke athoelke moved this to In Progress in PSA Certified API development Sep 12, 2025
Copy link
Copy Markdown
Contributor

@MarcusJGStreets MarcusJGStreets left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to be the obvious shape.

Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM except for a concern about HMAC.

I've only read the pull request and the NIST specification, I have never implemented Ascon.

Comment thread doc/crypto/api/ops/xof.rst Outdated
Comment thread doc/crypto/api/ops/hash.rst
Comment thread doc/crypto/api/ops/hash.rst Outdated
Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM at ea17af3 except that HMAC is still pending.

Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly LGTM but the HMAC block size discussion isn't quite right.

Comment thread doc/crypto/api/ops/mac.rst Outdated
Comment thread doc/crypto/api/ops/mac.rst Outdated
Comment thread doc/crypto/api/ops/mac.rst
Comment thread doc/crypto/api/ops/mac.rst
Comment thread doc/crypto/api/ops/mac.rst Outdated
Comment thread doc/crypto/api/ops/mac.rst Outdated
@athoelke
Copy link
Copy Markdown
Contributor Author

I think I've managed to remove the misleading discussion on 'small input-block sizes', and replace it with more accurate text.

Copy link
Copy Markdown
Contributor

@gilles-peskine-arm gilles-peskine-arm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@athoelke athoelke merged commit 2a630ee into ARM-software:main Oct 24, 2025
@github-project-automation github-project-automation Bot moved this from In Progress to Done in PSA Certified API development Oct 24, 2025
@athoelke athoelke deleted the crypto-ascon branch October 24, 2025 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

API design Related the design of the API Crypto API Issue or PR related to the Cryptography API enhancement New feature or request

Projects

Development

Successfully merging this pull request may close these issues.

Ascon Algorithm Identifiers

3 participants