add EIP-7702 transactions and receipts #575
Conversation
|
Please also add the 7702 tx to |
|
https://github.com/ethereum/EIPs/blob/master/EIPS/eip-7702.md The issue originally said 1559 + auth list, but EIP spec now says 4844 instead. "of the outer transaction follow the same semantics as EIP-4844." |
|
@lightclient can you review this plz hope I fixed your concern |
|
I think i messed up the branch with a bad rebase |
|
Plz move the conversation to #592 |
|
Please fix the branch here instead of opening a separate PR. It's important to keep the conversation around the change in the same thread. |
|
@lightclient resolved the issue thnks for the feedback and srry for skipping the issue and opening the new one |
Redidacove
force-pushed
the
eip_2718_txn_type
branch
from
de1ffbc to
030f607
Compare
There was a problem hiding this comment.
At Nethermind we just merged support for 7702 (NethermindEth/nethermind#7459) and we're currently extending our test suite. I have a couple of questions regarding this spec update and I would appreciate if you could take a look at them
lightclient
force-pushed
the
eip_2718_txn_type
branch
2 times, most recently
from
52439ef to
b45ec2e
Compare
There was a problem hiding this comment.
this looks complete to me
@lightclient on mekong there's currently v and yParity for this value, the EIP defines this as yParity, even though values >1 can technically be included in a (failing) transaction.
so imo naming this yParity, consistent with the EIP, makes the most sense
Co-authored-by: Lautaro Emanuel <[email protected]>
fjl
force-pushed
the
eip_2718_txn_type
branch
from
0f0de96 to
46762e5
Compare
|
Removed tests for now. I will add them back in #612 |
fjl
changed the title
Motivation
This is for closing #561
As mentioned I have added same eip 1559 + auth list referenced to reciept schema where added authlist definition