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

Hash to curve implementation #2

Open
kgncan opened this issue Nov 4, 2019 · 1 comment
Open

Hash to curve implementation #2

kgncan opened this issue Nov 4, 2019 · 1 comment

Comments

@kgncan
Copy link

kgncan commented Nov 4, 2019

Good work implementing this, and with sensible defaults.

One thing, I briefly looked at the implementation, and I'm not sure H' satisfies the requirements in the draft (which talks about using hash to curve algorithms from https://tools.ietf.org/html/draft-irtf-cfrg-hash-to-curve-04)

See:

// a = generator * password_n

This stack exchange answer implies that this might not be sufficient?

@louisabraham
Copy link

Do you think using 512 bits of output for the hash could solve the issue?
Also, wouldn't Universal Hashing require storing the random parameters somewhere?

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

No branches or pull requests

2 participants