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

chore: require destination for 7702 #1262

Merged
merged 4 commits into from
Sep 10, 2024
Merged

Conversation

mattsse
Copy link
Member

@mattsse mattsse commented Sep 9, 2024

The fields chain_id, nonce, max_priority_fee_per_gas, max_fee_per_gas, gas_limit, destination, value, data, and access_list of the outer transaction follow the same semantics as EIP-4844. Note, this means a null destination is not valid.

https://github.com/alloy-rs/eips/

to field is required for 7702

@mattsse mattsse force-pushed the matt/require-7702-to-field branch from 6cff529 to e99c973 Compare September 9, 2024 23:18
crates/rpc-types-eth/src/transaction/request.rs Outdated Show resolved Hide resolved
crates/consensus/src/transaction/eip7702.rs Outdated Show resolved Hide resolved
crates/consensus/src/header.rs Show resolved Hide resolved
mattsse and others added 2 commits September 10, 2024 01:19
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
@mattsse mattsse requested a review from DaniPopes September 9, 2024 23:23
@mattsse mattsse merged commit c9980cc into main Sep 10, 2024
30 checks passed
@mattsse mattsse deleted the matt/require-7702-to-field branch September 10, 2024 00:54
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.

2 participants