-
Notifications
You must be signed in to change notification settings - Fork 792
feat(debug_traceTransaction): initial commit #1469
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for this,
types needs some changes, otherwise this looks good.
also needs nightly formatting cargo +nightly fmt
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
Co-authored-by: Matthias Seitz <matthias.seitz@outlook.de>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
The |
Right, so the problem is that it requires an endpoint that supports |
Ah ok, in that case maybe let's disable it from the examples running in CI, or use Anvil for it (what's blocking us from supporting geth-style traces again in Anvil @mattsse?) ? |
next step is revm tracing conversion to geth types |
Motivation
Implementation of #1347
debug_traceTransaction
Solution
Relatively straightforward solution. The GethDebugOptions type was stolen from anvil.
PR Checklist
RPC_URL
environment variable is set to an RPC that supports debug_traceTransaction