Skip to content

ethapi: Fix DoEstimateGas to apply overrides with correct precompile set#31795

Merged
s1na merged 4 commits into
ethereum:masterfrom
MonkeyMarcel:DoEstimateGasFix
Aug 27, 2025
Merged

ethapi: Fix DoEstimateGas to apply overrides with correct precompile set#31795
s1na merged 4 commits into
ethereum:masterfrom
MonkeyMarcel:DoEstimateGasFix

Conversation

@MonkeyMarcel
Copy link
Copy Markdown
Contributor

fix and close #31719

Comment thread internal/ethapi/api.go
@MonkeyMarcel MonkeyMarcel requested a review from s1na May 17, 2025 05:41
@fjl fjl assigned s1na Jun 3, 2025
@MonkeyMarcel
Copy link
Copy Markdown
Contributor Author

@s1na Hi sina, sorry for the tagging, but this PR was put on hold for a while. Any modification should I make here?

Copy link
Copy Markdown
Contributor

@s1na s1na left a comment

Choose a reason for hiding this comment

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

LGTM, added a test

@s1na s1na added this to the 1.16.3 milestone Aug 27, 2025
@s1na s1na merged commit 7db6c91 into ethereum:master Aug 27, 2025
5 of 6 checks passed
gballet pushed a commit to gballet/go-ethereum that referenced this pull request Sep 11, 2025
…m#31795)

Fix and close ethereum#31719.

---------

Co-authored-by: Sina Mahmoodi <itz.s1na@gmail.com>
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.

DoEstimateGas doesn't allow precompile override

2 participants