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

Update deneb config vars #18

Merged
merged 2 commits into from
Nov 8, 2023
Merged

Update deneb config vars #18

merged 2 commits into from
Nov 8, 2023

Conversation

dapplion
Copy link
Member

@dapplion dapplion commented Nov 8, 2023

Complete deneb config with Gnosis custom values

MAX_BLOBS_PER_BLOCK

Preset MAX_BLOBS_PER_BLOCK is set to 6 to have the same block structure as Ethereum mainnet. Having a higher value than what execution allows will increase slightly the DoS vector for invalid blobs. However, now blob sidecars have an inclusion proof, so the vector is highly reduced.

With a MAX_BLOBS_PER_BLOCK higher than the max gas set in the execution spec, a malicious proposer could create a block referencing more blobs than possible. This block will be valid on consensus rules only. However, the execution node will reject the payload as its gas exceeds the maximum. A malicious proposer would therefore cause the network to consume ~3x more bandwidth than allowed, at the cost of missing a block proposal. Note that there's no free data availability, all nodes will eventually discard the block with too much blobs

MIN_EPOCHS_FOR_BLOB_SIDECARS_REQUESTS

MIN_EPOCHS_FOR_BLOB_SIDECARS_REQUESTS is increased to match the expected 2 weeks rollups consider today for Ethereum mainnet. The total disk requirement roughly equivalent to Ethereum mainnet since epochs are 2*12/5 faster

@dapplion dapplion merged commit b8ae309 into main Nov 8, 2023
1 check passed
@dapplion dapplion deleted the deneb-vars branch November 8, 2023 13:08
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