-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
BEP-441: Implement EIP-7702: Set EOA account code #2724
base: develop
Are you sure you want to change the base?
Conversation
6fb5abb
to
ca11f37
Compare
Co-authored-by: lightclient <lightclient@protonmail.com> Co-authored-by: Mario Vega <marioevz@gmail.com>
ca11f37
to
f6ca7a0
Compare
please reserve all commits when merging into develop! |
|
5pwMH5Tjcj-Pg-yy345 |
soeng.ll060@gmail.com> please reserve all commits when merging into develop! |
|
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.
listo
|
060331123
|
|
|
|
069331123 |
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.
convert to draft, wait the final spec |
Description
BEP-441: Implement EIP-7702: Set EOA account code
Rationale
tell us why we need these changes...
Example
add an example CLI or API response...
Changes
code is following ethereum/go-ethereum#30078
As said in (ethereum/go-ethereum#30519) , https://github.com/lightclient/go-ethereum/tree/prague-devnet-3 is is used for Pectra-devnet-3
in the prague upgrade, eip-7702 is the most complicated, and the spec still not be finalized.
according to https://notes.ethereum.org/@ethpandaops/pectra-devnet-3
run pectra-devnet-3@v1.5.0 against ethereum/go-ethereum#30078, still have many failed test cases 93/1596.
but after picking the code into bsc, there are 145/1596 failed cases.
by debugging, I find the reason, ethereum/go-ethereum#29520
but this PR is not easy to pick, so I do code implement after understanding it, outcome the second commit, please review it carefully! Now the number of passed test cases is 93/1596 too.