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

CIP4 - Wallet checksum #4

Merged

Conversation

SebastienGllmt
Copy link
Collaborator

@SebastienGllmt SebastienGllmt commented May 9, 2020

Discussion: https://forum.cardano.org/t/cip4-wallet-checksum/32819

This describes the wallet checksum that is implemented in the Yoroi Extension (and that will eventually be present in Yoroi Mobile). The goal of this CIP is to agree on the generation strategy for standardization with Daedalus in the future.

@SebastienGllmt SebastienGllmt requested a review from KtorZ May 9, 2020 21:16
@SebastienGllmt SebastienGllmt self-assigned this May 9, 2020

Although there is no specification for CRC32 and many variations exist, in Cardano we use the CRC-32-IEEE variation. You can find a C implementation [here](https://github.com/cardano-foundation/ledger-app-cardano/blob/3f784d23c1b87df73cda552ef01428d3e2733237/src/crc32.c#L6)

### Hash algorithms used in Shelley mainnet
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This part is mostly the part up for discussion. I'm open to any change for Shelley (since all users will get a new checksum anyway), or we can decide to keep it exactly the same if people want that

@SebastienGllmt SebastienGllmt changed the title CIP4 - Wallet checksum draft CIP4 - Wallet checksum May 10, 2020
@SebastienGllmt SebastienGllmt marked this pull request as draft May 10, 2020 01:11
@SebastienGllmt SebastienGllmt marked this pull request as ready for review September 22, 2020 07:48
@SebastienGllmt SebastienGllmt merged commit 863c9b8 into cardano-foundation:master Oct 27, 2020
@SebastienGllmt SebastienGllmt deleted the wallet-checksum branch October 27, 2020 20:32
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.

None yet

4 participants