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

fix(rpc): call, simulate, estimate rpcs executed on top of the block, not at the start of it #438

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cchudant
Copy link
Member

Pull Request type

  • Bugfix

What is the current behavior?

Call, simulate, estimate rpcs are now executed on top of the block they request instead of at the start of it.

What is the new behavior?

State changes that happen during the block requested in these rpc calls are now visible to the execution.
Trace block and trance transaction rpcs are unchanged - state changed during the block shouldn't be visible to these as they reexecute the requested block.

Does this introduce a breaking change?

No

@cchudant cchudant force-pushed the fix-rpc-exec-on-top-of-block branch 2 times, most recently from d26d5ce to 8b42c0a Compare December 23, 2024 11:52
@Trantorian1 Trantorian1 added bug Something isn't working full node Full Node only related issues labels Dec 31, 2024
@jbcaron jbcaron force-pushed the fix-rpc-exec-on-top-of-block branch from 8b42c0a to 6291a93 Compare January 5, 2025 21:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working full node Full Node only related issues
Projects
Status: In review
Development

Successfully merging this pull request may close these issues.

3 participants