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

debug_traceCall #8202

Open
1 task done
dexloom opened this issue May 10, 2024 · 1 comment · May be fixed by paradigmxyz/revm-inspectors#119
Open
1 task done

debug_traceCall #8202

dexloom opened this issue May 10, 2024 · 1 comment · May be fixed by paradigmxyz/revm-inspectors#119
Labels
A-rpc Related to the RPC implementation C-bug An unexpected or incorrect behavior

Comments

@dexloom
Copy link

dexloom commented May 10, 2024

Describe the bug

I believe debug_tracaCall in RETH doesn't return storage keys required . Not sure if the problem is in REVM or Tracers crate.

Here's a request that returns different data from GETH and RETH.

Inside the call, one of contract reverts, however calling contract handles this revert. Geth returns number of storage cells required for this call, but Reth doesn't.

Steps to reproduce

Try following request with RETH and GETH

I believe you can set block number instead of "latest"
serialized request request={"method":"debug_traceCall","params":[{"to":"0x9980ce3b5570e41324904f46a06ce7b466925e23","gas":"0xf4240","input":"0x2764cd0b000000000000000000000000352b186090068eb35d532428676ce510e17ab58100000000000000000000000000000000000000000000000009d09e34743bd41f000000000000000000000000000000000000000000000000000000000000000100000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000"},"latest",{"tracer":"prestateTracer","tracerConfig":{"diffMode":false}}],"id":8,"jsonrpc":"2.0"}

Node logs

No response

Platform(s)

Linux (x86)

What version/commit are you on?

reth Version: 0.2.0-beta.6-dev
Commit SHA: 2334317
Build Timestamp: 2024-05-02T07:47:43.588097501Z
Build Features: jemalloc
Build Profile: maxperf

What database version are you on?

Current database version: 2
Local database is uninitialized

Which chain / network are you on?

1 - Ethereum mainnet

What type of node are you running?

Archive (default)

What prune config do you use, if any?

No response

If you've built Reth from source, provide the full command you used

RUSTFLAGS="-C target-cpu=native" cargo build --profile maxperf --features jemalloc

Code of Conduct

  • I agree to follow the Code of Conduct
@dexloom dexloom added C-bug An unexpected or incorrect behavior S-needs-triage This issue needs to be labelled labels May 10, 2024
@mattsse mattsse added A-rpc Related to the RPC implementation and removed S-needs-triage This issue needs to be labelled labels May 12, 2024
@mattsse
Copy link
Collaborator

mattsse commented May 12, 2024

thanks, checking, must be in the prestate tracer logic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rpc Related to the RPC implementation C-bug An unexpected or incorrect behavior
Projects
Status: Todo
Development

Successfully merging a pull request may close this issue.

2 participants