-
Notifications
You must be signed in to change notification settings - Fork 135
supports state override #1893
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
supports state override #1893
Conversation
Signed-off-by: m4sterbunny <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
macfarla
left a comment
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.
This change looks fine.
When I searched the docs for "override", this is the first result: https://besu-docs-git-1722-consistent-syntax-clean-hyperledger.vercel.app/public-networks/how-to/send-transactions#override-state-values
so maybe that page should also have a link to the State Overrides object (and maybe mention the other RPCs, at least ethSimulateV1). Could be a separate PR.
Signed-off-by: m4sterbunny <[email protected]>
|
need to add eth_estimateGas to list on this page https://besu-docs-git-1874debugtracecall-options-hyperledger.vercel.app/public-networks/how-to/send-transactions#override-state-values otherwise LGTM |
Signed-off-by: m4sterbunny <[email protected]>
bgravenorst
left a comment
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
3988ea7 to
f681b0d
Compare
Description
Documents debug_traceCall State Overrides Support
Similar to eth_call - both methods use the same underlying mechanism to pass state overrides to the transaction simulator, allowing temporary state changes for testing and debugging without affecting the actual blockchain state.
Based on json in original pr, stateOverrides is nested inside the options object
Added debug_traceCall to the list of methods that accept state override objects
Issue(s) fixed
Fixes #1874
Preview