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

Transaction Leaf proof #5

Merged
merged 11 commits into from
Dec 1, 2023
Merged

Transaction Leaf proof #5

merged 11 commits into from
Dec 1, 2023

Conversation

nikkolasg
Copy link
Collaborator

Add transaction leaf proof circuitS:

  • first one decodes all headers sequentially
  • second one takes directly an offset and extract gas value directly.

There are potentially still many optimization to be done on the first one which is the only one really secure.

Copy link
Contributor

@lopeetall lopeetall left a comment

Choose a reason for hiding this comment

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

Everything looks great, except I'm confused about some functions in the test module in utils.rs. If you can explain what they are for and why they are placed where they are I can approve :)

src/mpt_tx.rs Show resolved Hide resolved
@lopeetall lopeetall merged commit 7a61d57 into main Dec 1, 2023
4 checks passed
@delehef delehef deleted the feat/tx_mpt.rs branch July 18, 2024 11:33
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