-
Notifications
You must be signed in to change notification settings - Fork 4
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
Clarify implemented Kyber version #33
Comments
Your assumption is correct, and I also recognized the need to follow the NIST draft (ML-KEM). |
The differences are listed in page 2 of the NIST draft 🙂 |
FYI I've implemented the ML-KEM changes here, in case you want to apply them to this repo too (I could also open a PR; we still need to maintain a separate fork to support browsers without native |
@larabr Thanks for your information. Although I'd like you to send a PR to merge your updates including the support for old browsers here, it is probably a tedious work for you, so I will try to merge them to some extent on my end. |
I assume this is implementing version 3 (as opposed to e.g. the NIST version), as that's stated by https://github.com/antontutoveanu/crystals-kyber-javascript . But I didn't find specific info about this.
Is the assumption correct? If so, do you have plans to implement the NIST draft version (https://csrc.nist.gov/pubs/fips/203/ipd)?
The text was updated successfully, but these errors were encountered: