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(storagext): losing transactions on reorg #651

Merged
merged 2 commits into from
Dec 20, 2024
Merged

Conversation

th7nder
Copy link
Contributor

@th7nder th7nder commented Dec 19, 2024

Description

There was still a time when a transaction was discarded during a Reorg.
This was because Mortality was set based on the Best block.
Best block was lost and transaction became invalid.

Fixes #647.

Checklist

  • Are there important points that reviewers should know?
    • If yes, which ones?
  • Make sure that you described what this change does.
  • Have you tested this solution?

@th7nder th7nder added the bug Something isn't working label Dec 19, 2024
@th7nder th7nder added this to the Phase 3 milestone Dec 19, 2024
@th7nder th7nder self-assigned this Dec 19, 2024
@th7nder th7nder added the ready for review Review is needed label Dec 19, 2024
@th7nder th7nder mentioned this pull request Dec 19, 2024
27 tasks
@cernicc cernicc merged commit ce0aa7f into develop Dec 20, 2024
5 checks passed
@cernicc cernicc deleted the fix/647/finalized-head branch December 20, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ready for review Review is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Transactions disappearing on a reorg
3 participants