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

add hash_to_curve method for Edwards curves #596

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

mikelodder7
Copy link
Contributor

Adds hash_to_curve for Edwards points according to RFC9390

@burdges
Copy link
Contributor

burdges commented Nov 7, 2023

RFC9390 requires an XoF, and gives sha2 xmd as a named cipher suite, but RFC9390 permits choosing another XoF like shake128 instead. I guess maybe the traits expose this option?

@mikelodder7
Copy link
Contributor Author

Yes they do. If you look ExpandMsg supports ExpandMsgXmd or ExpandMsgXof

@burdges
Copy link
Contributor

burdges commented Nov 7, 2023

Alright, so already possible to choose shake128 with this PR? nice :)

@mikelodder7
Copy link
Contributor Author

Yes any XOF, Shake128, Shake256, etc.

Signed-off-by: Michael Lodder <[email protected]>
@dignifiedquire
Copy link
Contributor

Fyi it is RFC 9380 not 9390 :)

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.

3 participants