-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
contracts-bedrock: add size limit on deposit transaction size #5527
Conversation
|
✅ Deploy Preview for opstack-docs canceled.
|
Hey @tynes! This PR has merge conflicts. Please fix them before continuing review. |
4a44eb8
to
96aeb8a
Compare
Hey @tynes! This PR has merge conflicts. Please fix them before continuing review. |
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.
This is g2g after conflicts are resolved.
This PR has been added to the merge queue, and will be merged soon. |
Hey @tynes, this pull request failed to merge and has been dequeued from the merge train. If you believe your PR failed in the merge train because of a flaky test, requeue it by commenting with |
a7982f3
to
6fd89d1
Compare
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.
GTG
Hey @tynes! This PR has merge conflicts. Please fix them before continuing review. |
Adds a limit to prevent calldata larger than 120kb. Unsafe blocks contain the deposit transactions and when there is no upper bound on the calldata size, it is possible for a single transaction to cause the p2p gossip to consume far more bandwidth than necessary.
6fd89d1
to
fb2adc4
Compare
Description
Adds a limit to prevent calldata larger than 120kb. Unsafe blocks contain the deposit transactions and when there is no upper bound on the calldata size, it is possible for a single transaction to cause the p2p gossip to consume far more bandwidth than necessary.
#5529 should be merged first because of the
semver
bumpingTests
Includes test coverage of the new revert condition
Fixes CLI-3847