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

Implement tagged hashes for the leafhashes #118

Conversation

kcalvinalvin
Copy link
Contributor

For versioning purposes, we want to prepend the preimage with a tag. There's more efficient ways to do this but that requires modifying rust-bitcoin and this is just the most straightforward way to implement tagged hashes.

@kcalvinalvin kcalvinalvin force-pushed the 2024-01-10-super-basic-tagged-hash-impl branch from ec37474 to d25be5c Compare January 11, 2024 07:55
For versioning purposes, we want to prepend the preimage with a tag.
There's more efficient ways to do this but that requires modifying
rust-bitcoin and this is just the most straightforward way to implement
tagged hashes.
@kcalvinalvin kcalvinalvin force-pushed the 2024-01-10-super-basic-tagged-hash-impl branch from d25be5c to 3462e50 Compare January 11, 2024 08:07
@kcalvinalvin
Copy link
Contributor Author

kcalvinalvin commented Jan 11, 2024

Fixed all the bugs and verified that the generated roots are the same as utreexod by using the same blocks.txt file in crates/floresta-chain/src/pruned_utreexo/testdata/

Relevant PR here:
utreexo/utreexod#112

@a-moreira
Copy link
Collaborator

Concept ACK

@Davidson-Souza Davidson-Souza merged commit a648209 into vinteumorg:master Jan 12, 2024
6 checks passed
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.

3 participants