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

Implement Calltracer in anvil #6882

Open
mattsse opened this issue Jan 22, 2024 · 2 comments · May be fixed by #6884
Open

Implement Calltracer in anvil #6882

mattsse opened this issue Jan 22, 2024 · 2 comments · May be fixed by #6884
Labels
good first issue Good for newcomers T-feature Type: feature

Comments

@mattsse
Copy link
Member

mattsse commented Jan 22, 2024

Component

Anvil

Describe the feature you would like

currently the tracer type callTracer is ignored as highlighted by #6873

this isn't handled properly:

pub fn geth_trace(&self, opts: GethDefaultTracingOptions) -> DefaultFrame {
GethTraceBuilder::new(self.info.traces.clone(), TracingInspectorConfig::default_geth())
.geth_traces(
self.receipt.gas_used().as_u64(),
self.info.out.clone().unwrap_or_default().0.into(),
opts,
)
}

see for ref:

https://github.com/paradigmxyz/reth/blob/021d236f8c7a067cfbcc4ecd7ff77b6a72adc1cb/crates/rpc/rpc/src/debug.rs#L473-L473

Additional context

No response

@qiweiii
Copy link
Contributor

qiweiii commented Jan 23, 2024

hi @mattsse I put up a draft pr, a question is do we only need to add CallTracer or match all types of GethDebugTracerType like in https://github.com/paradigmxyz/reth/blob/021d236f8c7a067cfbcc4ecd7ff77b6a72adc1cb/crates/rpc/rpc/src/debug.rs#L473-L473

@prasincs
Copy link

@qiweiii sweet, I can help test this and any subsequent DebugTracer types, but I'm most interested in calltracer right now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers T-feature Type: feature
Projects
No open projects
Status: Todo
3 participants