-
Notifications
You must be signed in to change notification settings - Fork 38
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
Standardise L1 block processing #2164
base: main
Are you sure you want to change the base?
Conversation
Nice, I like the idea of preparing all the TEN-relevant block data on the host side and then passing it in with the block.
|
Actually one other concern I had, which I think is why we started doing this enclave side to begin with is that it's easy enough to prove TEN-data was found in that block, but it's not easy to prove that the host didn't miss anything. Maybe the incentives are such that there aren't any useful attacks there. If the host misses a x-chain msg or a sequencer-permissioning message then the enclave would get into a bricked state where it could no longer process new batches but that's not v useful as an attack I imagine. |
Why this change is needed
https://github.com/ten-protocol/ten-internal/issues/4299
What changes were made as part of this PR
TODO
PR checks pre-merging
Please indicate below by ticking the checkbox that you have read and performed the required
PR checks