Skip to content

fix(spec-specs): load the recipient after authorization processing - #3133

Closed
spencer-tb wants to merge 1 commit into
ethereum:forks/amsterdamfrom
spencer-tb:eips/amsterdam/eip-7928-bal-recipient-halt
Closed

fix(spec-specs): load the recipient after authorization processing#3133
spencer-tb wants to merge 1 commit into
ethereum:forks/amsterdamfrom
spencer-tb:eips/amsterdam/eip-7928-bal-recipient-halt

Conversation

@spencer-tb

@spencer-tb spencer-tb commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Description

Per EIPs#11902, the recipient must not enter the block access list when a halt during EIP-7702 authorization processing precedes its load. Defer the recipient code load from prepare_message to after set_delegation; behavior-neutral today (authorization costs are intrinsic, so no halt can intervene), it keeps the recipient unread once EIP-2780 charges them at runtime (#3126).

Related Issues or PRs

Implements EIPs#11902; ordering becomes observable with #3126.

Checklist

  • Ran fast static checks to avoid CI fails, see Code Standards & Verifying Changes: just static
  • PR title has the form <type>(<area>): <title>, where <type> and <area> come from an appropriate C-<type>, respectively A-<area>, label. The title should match the target squash commit message.

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@spencer-tb spencer-tb added A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) C-bug Category: this is a bug, deviation, or other problem A-tests Area: Consensus tests. labels Jul 9, 2026
@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.30%. Comparing base (78daabc) to head (36e5056).

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #3133   +/-   ##
================================================
  Coverage            93.30%   93.30%           
================================================
  Files                  624      624           
  Lines                36991    36992    +1     
  Branches              3384     3384           
================================================
+ Hits                 34515    34516    +1     
  Misses                1694     1694           
  Partials               782      782           
Flag Coverage Δ
unittests 93.30% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@spencer-tb
spencer-tb force-pushed the eips/amsterdam/eip-7928-bal-recipient-halt branch from 89d15f3 to 36e5056 Compare July 9, 2026 15:00
@spencer-tb spencer-tb changed the title fix(spec-specs, tests): exclude unloaded recipient from the BAL fix(spec-specs): load the recipient after authorization processing Jul 9, 2026
@spencer-tb spencer-tb closed this Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-spec-specs Area: Specification—The Ethereum specification itself (eg. `src/ethereum/*`) A-tests Area: Consensus tests. C-bug Category: this is a bug, deviation, or other problem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant