Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Specify 'logs root' property of L2 Blocks #45

Closed
maurelian opened this issue Dec 1, 2021 · 2 comments
Closed

Specify 'logs root' property of L2 Blocks #45

maurelian opened this issue Dec 1, 2021 · 2 comments

Comments

@maurelian
Copy link
Collaborator

We should consider including a "logs root" (something more neat than the receipt root on L1).
Merkleizing the L1 log events more efficiently, filtered to only those of the successful receipts, would be a nice thing to do as rollup. If L2 doesn't pay for a log-subscription in advance (the extension we talked about), at least it can efficiently provide a proof later on that something happened on L1.

This issue is a placeholder, taken from a PR comment here by @protolambda.

@maurelian maurelian mentioned this issue Dec 2, 2021
3 tasks
@norswap
Copy link
Contributor

norswap commented Dec 2, 2021

Is this worth it? The merkleized entries in the receipts root comprise a concatenation of all log events emitted during a transaction. Seems like except extremely weird cases, this should not be too big? EIP-4488 will also drastically reduce the cost of proving the entry.

@protolambda
Copy link
Collaborator

It's not about the cost of the fraud-proof here, but about the cost for a L2 user inside the L2 EVM to casually proof some L1 event.

If we did the heavy lifting by providing a nice to work with accumulator then L2 users will have to pay less fees and it may also be less error prone for them (a lot easier to verify a plain byte array of a log entry in a binary tree, than to verify a RLP-encoded receipt in a MPT)

@ethereum-optimism ethereum-optimism locked and limited conversation to collaborators Mar 9, 2022
@trianglesphere trianglesphere converted this issue into discussion #228 Mar 9, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Projects
None yet
Development

No branches or pull requests

4 participants