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.

Sign up for GitHub

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

Jump to bottom

fix: wrong returned gas used and handler crash in debug APIs #276

Merged
merged 2 commits into from
May 24, 2023
Merged

fix: wrong returned gas used and handler crash in debug APIs #276

merged 2 commits into from
May 24, 2023

Commits on May 17, 2023

  1. fix: wrong returned gas used of system transaction in debug APIs 

    When using debug APIs, system transaction goes through ApplyMessage which
currently accounts for instrinsicGas and refund while normal flow through
Finalize/FinalizeAndAssemble does not. This commit adds an IsSystemTransaction to
evm's config, when system transaction goes through ApplyMessage, turn on this
flag so that gasUsed does not account for intrinsicGas and refund.
    @minh-bq
    minh-bq committed May 17, 2023
    Configuration menu
    Copy the full SHA
    5f2aaa3 View commit details
    Browse the repository at this point in the history

Commits on May 24, 2023

  1. eth/tracers: fix flatCallTracer crasher (#27304) 

    FlatCallTracer had a crasher when it was passed `onlyTopCall: true` as config.
This PR ignores config fields inherited from the normal call tracer.
    @minh-bq
    minh-bq committed May 24, 2023
    Configuration menu
    Copy the full SHA
    85e3d21 View commit details
    Browse the repository at this point in the history