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

turbo/jsonrpc: add optional includePrecompiles flag to trace_* apis #11023

Merged
merged 1 commit into from
Jul 4, 2024

Conversation

taratorio
Copy link
Member

cherry pick: 9710e98
for E3


relates to #9784

  • Adds support for an optional "includePrecompiles" tracer config option for our OeTracer (OpenEthereum) that users can use to match output of debug_* apis with callTracer (by default it includes precompiles). Note default spec for OpenEthereum traces are to not include precompiles - this is preserved by this PR
  • Note geth has support for "includePrecompiles" so we are getting more aligned as well -
    https://github.com/ethereum/go-ethereum/blob/master/eth/tracers/native/call_flat.go#L124
  • Adds tests for OeTracer

…10979)

relates to #9784

- Adds support for an optional `"includePrecompiles"` tracer config
option for our OeTracer (OpenEthereum) that users can use to match
output of debug_* apis with callTracer (by default it includes
precompiles). Note default spec for OpenEthereum traces are to not
include precompiles - this is preserved by this PR
- Note geth has support for `"includePrecompiles"` so we are getting
more aligned as well -
https://github.com/ethereum/go-ethereum/blob/master/eth/tracers/native/call_flat.go#L124
- Adds tests for OeTracer
@taratorio taratorio added this to the 2.60.3-fixes milestone Jul 4, 2024
@VBulikov VBulikov self-requested a review July 4, 2024 11:40
@taratorio taratorio enabled auto-merge (squash) July 4, 2024 11:48
@taratorio taratorio merged commit 2711926 into main Jul 4, 2024
10 checks passed
@taratorio taratorio deleted the e3-oe-tracer-incl-precompiles-flag branch July 4, 2024 12:02
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