Skip to content

internal/ethapi: include AuthorizationList in gas estimation callArgs#33849

Merged
s1na merged 1 commit intoethereum:masterfrom
vickkkkkyy:AuthorizationList
Feb 23, 2026
Merged

internal/ethapi: include AuthorizationList in gas estimation callArgs#33849
s1na merged 1 commit intoethereum:masterfrom
vickkkkkyy:AuthorizationList

Conversation

@vickkkkkyy
Copy link
Copy Markdown
Contributor

The callArgs constructed for gas estimation in setDefaults copies AccessList, BlobHashes, etc. but misses AuthorizationList. This means SetCode (EIP-7702) transactions without an explicit gas value get estimated without their authorization list, leading to incorrect gas estimates.

Copy link
Copy Markdown
Member

@MariusVanDerWijden MariusVanDerWijden left a comment

Choose a reason for hiding this comment

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

SGTM

@vickkkkkyy
Copy link
Copy Markdown
Contributor Author

SGTM

Greeting! is this good to process?

@s1na s1na added this to the 1.17.1 milestone Feb 23, 2026
@s1na s1na merged commit 1625064 into ethereum:master Feb 23, 2026
7 of 8 checks passed
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Feb 25, 2026
AnilChinchawale pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Feb 28, 2026
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.

3 participants