-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: send l1infotree
data to the aggchain prover
#183
feat: send l1infotree
data to the aggchain prover
#183
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is the reason that you need blockHash to all syncers? It looks like is not used on bridesync neither lastgersync.. isn't?
To be consistent. It is useful for debugging, if we have issues on other syncers with reorg detection. |
|
Description
This PR implements the additional logic of sending
l1infotree
data to theaggchain prover
that is needed to generate aaggchain
(auth
) proof.The data includes:
l1InfoTreeRoot
.l1infoTreeRoot
.Based on the provided block range and on the finalized l1 info tree data we send to the prover, prover can either return the maximum block range we asked, or return a smaller range.
But, in order to do this, all the syncers need to store block hash as well to the block table. Before this PR, only
l1infotreesyncer
stored this data, but we might need it in all syncers (to be consistent), so this PR expanded all the syncers to store the block hash as well.Fixes # (issue)