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

Change names of algorithms from Star Wars-based to NIST-based #46

Merged
merged 3 commits into from
May 6, 2024

Conversation

kriskwiatkowski
Copy link
Contributor

@kriskwiatkowski kriskwiatkowski commented Mar 16, 2024

Changes names from Kyber, Dilithium, Sphincs+ to ML-KEM, ML-DSA, SLH-DSA

@auriee As discussed.

Closes #28

@kriskwiatkowski kriskwiatkowski force-pushed the kris/fipsify branch 2 times, most recently from 679147f to 1976295 Compare March 16, 2024 10:59
@kriskwiatkowski kriskwiatkowski changed the title Change names of algorithms from Star Wars based to NIST based Change names of algorithms from Star Wars-based to NIST-based Mar 16, 2024
| 2 | SHA-256/SHA3-256 (collision search) | Dilithium2 |
| 3 | AES-192 (exhaustive key recovery) | Kyber768, Dilithium3, Sphincs+SHA-256 192f/s |
| 1 | AES-128 (exhaustive key recovery) | ML-KEM-512, FN-DSA-512, SLH-DSA-SHA2/SHAKE-128f/s |
| 2 | SHA-256/SHA3-256 (collision search) | ML-DSA-65 |
Copy link
Owner

@tireddy2 tireddy2 May 1, 2024

Choose a reason for hiding this comment

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

For PQ Security Level 2, it is ML-DSA-44. ML-DSA-65 is specified both at level 2 and 3.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh,crap. sorry for that. Let me fix it right away.

Copy link
Owner

@tireddy2 tireddy2 left a comment

Choose a reason for hiding this comment

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

Rest of the changes look good.

@tireddy2 tireddy2 merged commit 2359453 into tireddy2:main May 6, 2024
1 check passed
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.

Name change for Kyber, Dilithium and SPHINCS+
2 participants