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 EIP-7623: Move to Withdrawn #8530

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

wjmelements
Copy link
Contributor

Changes

  • Describe Gas Sheltering in Security Considerations section
  • Move status to Withdrawn

@wjmelements wjmelements requested a review from eth-bot as a code owner May 8, 2024 02:09
@github-actions github-actions bot added c-status Changes a proposal's status s-withdrawn This EIP is Withdrawn t-core labels May 8, 2024
@eth-bot
Copy link
Collaborator

eth-bot commented May 8, 2024

File EIPS/eip-7623.md

Requires 1 more reviewers from @nerolation, @vbuterin

@eth-bot eth-bot added the a-review Waiting on author to review label May 8, 2024
@eth-bot eth-bot changed the title Update EIP-7623: Disclose Gas Sheltering Security Issue Update EIP-7623: Move to Withdrawn May 8, 2024
@nerolation
Copy link
Contributor

I disagree.
You say in the security section that one needs to consider the worst case blocks after users start playing the games you describe:

Have you ever checked what the worst case block size would then be?
If there's a user that needs EVM computation worth 20m gas and another who wants to add 10m gas worth of data to the chain, then, depite being irrelevant if it's one or two users, the max possible blocksize is still significantly reduced.

@wjmelements
Copy link
Contributor Author

wjmelements commented May 8, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-review Waiting on author to review c-status Changes a proposal's status s-withdrawn This EIP is Withdrawn t-core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants