Skip to content
This repository has been archived by the owner on Jul 15, 2018. It is now read-only.

ed25519 addresses: remove amino, use SHA256[:20] #103

Closed
ebuchman opened this issue May 22, 2018 · 1 comment
Closed

ed25519 addresses: remove amino, use SHA256[:20] #103

ebuchman opened this issue May 22, 2018 · 1 comment

Comments

@ebuchman
Copy link
Contributor

ebuchman commented May 22, 2018 •
edited
Loading

This causes pain in #77 and is included in tendermint/tendermint#1568

Let's make the PubKeyEd25519.Address() return the first 20 bytes of the hash of the raw 32-byte pubkey, no amino required

See new tmhash for the hash function: tendermint/tmlibs#210
This was referenced May 22, 2018
Closed
Closed
@ebuchman ebuchman mentioned this issue May 30, 2018
Merged
@liamsi
Copy link
Contributor

liamsi commented Jun 12, 2018

resolved via #112

@liamsi liamsi closed this as completed Jun 12, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ebuchman @liamsi