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

bug(mempool): correctly handle tx from same account #442

Open
1 task done
Tracked by #419
HermanObst opened this issue Dec 26, 2024 · 0 comments · May be fixed by #450
Open
1 task done
Tracked by #419

bug(mempool): correctly handle tx from same account #442

HermanObst opened this issue Dec 26, 2024 · 0 comments · May be fixed by #450
Assignees
Labels
bug Something isn't working sequencer The label for sequencing related issues

Comments

@HermanObst
Copy link

HermanObst commented Dec 26, 2024

Is there an existing issue?

  • I have searched the existing issues

Description of bug

When the mempool receives multiple transactions from the same account, it currently fails to handle cases where a transaction with nonce (x+1) arrives before the prerequisite transaction with nonce (x). This results in an "invalid nonce" error for the (x+1) transaction.

Expected Behavior:
The transaction with nonce (x+1) should remain in the mempool in a pending state until the transaction with nonce (x) is received and successfully executed. Once the (x) transaction completes, the system should automatically process the waiting (x+1) transaction in the correct sequence.

Steps to reproduce

  1. Identify current nonce (n) for a account (maybe a devnet account)
  2. Submit a transaction with nonce (n+1)
  3. Observe immediate rejection instead of queuing
@Trantorian1 Trantorian1 self-assigned this Jan 1, 2025
@Trantorian1 Trantorian1 added bug Something isn't working sequencer The label for sequencing related issues labels Jan 1, 2025
@Trantorian1 Trantorian1 moved this to In progress in Madara Jan 1, 2025
@Trantorian1 Trantorian1 linked a pull request Jan 1, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working sequencer The label for sequencing related issues
Projects
Status: In progress
Development

Successfully merging a pull request may close this issue.

2 participants