Skip to content

Conversation

@0xmountaintop
Copy link

@0xmountaintop 0xmountaintop commented Mar 15, 2022

also update in notion doc

@ChuhanJin
Copy link

Can one of the admins verify this patch?

@noel2004
Copy link
Member

noel2004 commented Mar 15, 2022

LGTM. But I guess we still need the roothash of account trie to complete the information which MPT circuit needed. May be we can add one more field in the extraData structure?

sorry this may not work because calling GetProof from the account trie just access the state before tx begins... account trie is not updated among execution

@0xmountaintop
Copy link
Author

closed in favor of #46

@0xmountaintop 0xmountaintop deleted the trace/storage/account_proof branch March 18, 2022 01:48
pdrobnjak pushed a commit to Tenderly/net-scroll-geth that referenced this pull request Apr 30, 2025
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.

5 participants