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

eth/tracers: apply latest tracers from ethereum #270

Merged
merged 6 commits into from
Apr 28, 2023

Conversation

DNK90
Copy link
Contributor

@DNK90 DNK90 commented Apr 27, 2023

Purposes

Our tracers seems outdated since v1.10.3. There are a lots of update from this version. Therefore, I made this PR to update all changes from v1.10.4 to latest tracers PR (#27031)

Important Updates

  • Upgrade go version from 1.17 to 1.20
  • Support prestate with diffMode, multiplexing tracer (muxTracer) and flatCallTracer (aka parity style tracer)
  • Replace duktape to goja to handle JS tracer
  • Support block overrides in debug_traceCall
  • Include revertReason to callTracer
  • Add onlyTopCall and withLog options to callTracer
  • Replace opcode 0xfe to INVALID opcode

@DNK90 DNK90 requested review from minh-bq and linh-1 April 27, 2023 04:09
eth/tracers/api.go Show resolved Hide resolved
eth/tracers/api.go Show resolved Hide resolved
@DNK90 DNK90 force-pushed the chore/update_latest_tracers branch from 273582f to a8fecfc Compare April 27, 2023 10:36
@DNK90 DNK90 requested a review from minh-bq April 27, 2023 10:56
@DNK90 DNK90 merged commit f5b915e into master Apr 28, 2023
@DNK90 DNK90 deleted the chore/update_latest_tracers branch April 28, 2023 02:11
andicrypt pushed a commit to andicrypt/ronin that referenced this pull request Nov 1, 2023
* chore: apply latest tracers from ethereum

* chore: update go version in Dockerfile and ci

* fix: TestClientSyncTree failed

* fix: TestParseEntry failed

* p2p/dnsdisc: fix tests with Go 1.20 (#26690)

* eth/tracers/api: apply parallel tracing for `traceInternalsAndAccounts`

---------

Co-authored-by: Felix Lange <fjl@twurst.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.

3 participants