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

We don't cover all re-org cases (dispute tx reorged) #241

Open
mariocynicys opened this issue Sep 18, 2023 · 0 comments
Open

We don't cover all re-org cases (dispute tx reorged) #241

mariocynicys opened this issue Sep 18, 2023 · 0 comments

Comments

@mariocynicys
Copy link
Collaborator

We should handle trackers which are marked as InMempoolSince but their disputes have been reorged.
Right now we only handle trackers which are confirmed and then reorged, we don't consider their disputes nor the disputes of in-mempool trackers.

We should make sure the dispute is confirmed as well.

// TODO: Not only confirmed trackers need to be marked as reorged, but trackers that hasn't confirmed but their
// dispute did confirm in the reorged block. We can pull dispute txids of non confirmed penalties and get their
// confirmation block from our tx_index.
self.reorged_trackers.lock().unwrap().extend(

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

No branches or pull requests

1 participant