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

Consensus: 2nd tx should not finish before 1st tx #661

Open
kstroobants opened this issue Dec 3, 2024 · 0 comments · May be fixed by #759
Open

Consensus: 2nd tx should not finish before 1st tx #661

kstroobants opened this issue Dec 3, 2024 · 0 comments · May be fixed by #759
Assignees

Comments

@kstroobants
Copy link
Contributor

kstroobants commented Dec 3, 2024

For failed appeals close to the end of the finality window

  1. tx1 accepted and tx2 accepted
  2. tx1 processing appeal
  3. tx2 should not be finalized

Quick test in studio:

  1. Set finality window on 30s
  2. Send tx1 and tx to deployed contract
  3. Wait until both are in accepted state
  4. Appeal tx1
  5. Set finality window to 3s
  6. Check if tx2 is finalized after tx1
@kstroobants kstroobants self-assigned this Dec 17, 2024
@kstroobants kstroobants linked a pull request Dec 17, 2024 that will close this issue
4 tasks
@kstroobants kstroobants linked a pull request Dec 17, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant