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

EIP-4762: Statelessness gas cost changes #4762

Merged
merged 10 commits into from
Mar 27, 2022

Conversation

dankrad
Copy link
Contributor

@dankrad dankrad commented Feb 3, 2022

This EIP introduces changes in the gas schedule to reflect the costs of creating a witness. It requires clients to update their database layout to match this, so as to avoid potential DoS attacks.

@dankrad dankrad mentioned this pull request Feb 3, 2022
@eth-bot
Copy link
Collaborator

eth-bot commented Feb 3, 2022

All tests passed; auto-merging...

(pass) eip-4762.md

classification
updateEIP
  • passed!

Copy link
Contributor

@MicahZoltu MicahZoltu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Along with inline comments, please include the following sections at the end:

## Security Considerations
All EIPs must contain a section that discusses the security implications/considerations relevant to the proposed change. Include information that might be important for security discussions, surfaces risks and can be used throughout the life cycle of the proposal. E.g. include security-relevant design decisions, concerns, important discussions, implementation-specific guidance and pitfalls, an outline of threats and risks and how they are being addressed. EIP submissions missing the "Security Considerations" section will be rejected. An EIP cannot proceed to status "Final" without a Security Considerations discussion deemed sufficient by the reviewers.

## Copyright
Copyright and related rights waived via [CC0](https://creativecommons.org/publicdomain/zero/1.0/).

EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
dankrad and others added 4 commits February 10, 2022 11:27
Co-authored-by: Micah Zoltu <micah@zoltu.net>
Co-authored-by: Micah Zoltu <micah@zoltu.net>
Co-authored-by: Micah Zoltu <micah@zoltu.net>
Co-authored-by: Micah Zoltu <micah@zoltu.net>
Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This EIP is also missing the Security Considerations section and Copyright section.

EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
Copy link
Contributor

@MicahZoltu MicahZoltu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a few changes needed to satisfy CI.

EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
EIPS/eip-4762.md Outdated Show resolved Hide resolved
@eth-bot eth-bot enabled auto-merge (squash) March 26, 2022 08:17
Co-authored-by: Micah Zoltu <micah@zoltu.net>
auto-merge was automatically disabled March 26, 2022 09:06

Head branch was pushed to by a user without write access

dankrad and others added 2 commits March 26, 2022 09:06
Co-authored-by: Micah Zoltu <micah@zoltu.net>
Co-authored-by: Micah Zoltu <micah@zoltu.net>
@eth-bot eth-bot enabled auto-merge (squash) March 26, 2022 09:07
EIPS/eip-4762.md Outdated Show resolved Hide resolved
@eth-bot eth-bot merged commit bff0154 into ethereum:master Mar 27, 2022
PowerStream3604 pushed a commit to PowerStream3604/EIPs that referenced this pull request May 19, 2022
* Verkle gas cost changes initial commit

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Add sections, move discussion to magicians

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>

* Update EIPS/eip-4762.md

Co-authored-by: Micah Zoltu <micah@zoltu.net>
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.

4 participants