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

Invalid proof of election in self-repair #287

Closed
ghost opened this issue Apr 26, 2022 · 1 comment
Closed

Invalid proof of election in self-repair #287

ghost opened this issue Apr 26, 2022 · 1 comment
Labels
bug Something isn't working self repair Involve SelfRepair mechanism to-analyze

Comments

@ghost
Copy link

ghost commented Apr 26, 2022

What happened?

image

This issue happens when a node was running for quite some time, and then a new node joins the network.
When this new node tries to synchronize all the missing transactions, it throws errors saying the proof of election is invalid.

This is probably an issue with ordering of transactions

What is expected ?

When a node joins the network, it should download and synchronize transactions sequentially to ensure the consistency and the order of the transactions and in-memory states.


Epic: #321

@ghost ghost added bug Something isn't working self repair Involve SelfRepair mechanism labels Apr 26, 2022
@ghost ghost changed the title Invalid proof of election Invalid proof of election in self-repair Apr 26, 2022
@ghost
Copy link
Author

ghost commented Apr 27, 2022

Turns out, beacon summary aggregates contains transaction summaries which are after the summary time.
Which means others beacon summary for other time later will come, and the transactions will not be in the right order.

image

There is a chance the BeaconChain.get_full_beacon_summary is causing the issue.

@ghost ghost added the to-analyze label Apr 27, 2022
@ghost ghost self-assigned this Apr 27, 2022
@ghost ghost mentioned this issue Apr 29, 2022
@ghost ghost closed this as completed May 2, 2022
@ghost ghost mentioned this issue May 13, 2022
@ghost ghost mentioned this issue Jun 27, 2022
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working self repair Involve SelfRepair mechanism to-analyze
Projects
None yet
Development

No branches or pull requests

0 participants