Skip to content

Sparse merkle tree multi-proof#555

Closed
hujw77 wants to merge 1 commit intoSnowfork:mainfrom
hujw77:main
Closed

Sparse merkle tree multi-proof#555
hujw77 wants to merge 1 commit intoSnowfork:mainfrom
hujw77:main

Conversation

@hujw77
Copy link

@hujw77 hujw77 commented Dec 21, 2021

For gas saving, we could use the sparse merkle tree multi-proof to validate the commitment signed by beefy authority set.
More detail can see darwinia-network/darwinia-messages-sol#104

@vgeddes vgeddes requested review from musnit and vgeddes December 22, 2021 11:51
@vgeddes
Copy link
Collaborator

vgeddes commented Dec 22, 2021

Hey @hujw77 thanks for the contribution. I'll be able to review this properly next week, unless my colleagues get to it first.

@musnit
Copy link
Collaborator

musnit commented Jan 12, 2022

@hujw77 Taking another quick look - Using a multiproof instead of an array of individual proofs looks like a great idea and optimization!

We'll need to build the proof creation logic into our relayer and do a deeper team audit and review before merging this. At the moment we're focused on some other maintenance and new feature work, so it may take a few weeks before we schedule this work in, but will leave this open as it's definitely something we'll want to merge in before this code is used in production.

Thanks again!

@hujw77
Copy link
Author

hujw77 commented Jan 13, 2022

@musnit For the proof creation logic, there is research about it at https://github.com/hujw77/sparse-merkle-tree
It may be some help for audit and review.

@vgeddes
Copy link
Collaborator

vgeddes commented Jan 12, 2023

@vgeddes vgeddes closed this Jan 12, 2023
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