-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Update ed25519 backend #7890
Comments
@marbar3778 - is Tendermint v0.35 scheduled for SDK 0.40.1 or 0.41.0? |
Does the sdk have a timetable for 0.41. It's hard to say when 0.35 will be ready |
We have a task list https://github.com/cosmos/cosmos-sdk/milestone/26 :D |
Yea it should be compatible. I dont think the sdk does any key verification for tm ed25519 keys? If this is the case, it is safe to make this update now. |
I'm not aware about any such place. @aaronc - do you know if we do that ? |
There is also a security discussion about ed25519 in #4789 |
Summary
Tendermint v0.35 (scheduled post Stargate) changed ed25519 backend to according to zip215
Details: tendermint/tendermint#5632
Problem Definition
For compatibility with other libraries we need to have more strict signature verification than the default one provided by stdlib.
Proposal
Simple task: use github.com/hdevalence/ed25519consensus (instead of stdlib) for ed25519 signature verification.
For Admin Use
The text was updated successfully, but these errors were encountered: