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.

Sign up for GitHub

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

Jump to bottom

common/crypto/ed25519: Use curve25519-voi #3829

Closed
3 tasks done
Yawning opened this issue Apr 1, 2021 · 1 comment
Closed
3 tasks done

common/crypto/ed25519: Use curve25519-voi #3829

Yawning opened this issue Apr 1, 2021 · 1 comment
Assignees
@Yawning

Comments

@Yawning
Copy link
Contributor

Yawning commented Apr 1, 2021 •
edited
Loading

Once it has been reviewed by a third party, we should switch from our old ed25519 implementation to curve25519-voi, because verification is faster (Before the peanut gallery suggests ed25519consensus, curve25519-voi is significantly faster on modern Intel systems).
@Yawning
Copy link
Contributor Author

Yawning commented Jun 28, 2021

This will happen the next time we update our tendermint fork to something more recent, so I'm ok with closing this.

@Yawning Yawning closed this as completed Jun 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Yawning Yawning

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Yawning