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

Reorg ropsten solution #2419

Merged
merged 15 commits into from
Jul 22, 2021
Merged

Reorg ropsten solution #2419

merged 15 commits into from
Jul 22, 2021

Conversation

AlexeyAkhunov
Copy link
Contributor

No description provided.

@MariusVanDerWijden
Copy link
Contributor

Just fyi, I'm already reorging ropsten, for geth you would just need to full sync with the fixed version (which rejects the bad block) and then mine on top. I'm at 10,682,071 which is ~2000 blocks ahead from the bad block. But the bad chain keeps on advancing too

@AlexeyAkhunov
Copy link
Contributor Author

Just fyi, I'm already reorging ropsten, for geth you would just need to full sync with the fixed version (which rejects the bad block) and then mine on top. I'm at 10,682,071 which is ~2000 blocks ahead from the bad block. But the bad chain keeps on advancing too

Thank you! The reason I am doing this PR is to make sure we have tools to do something like this on the main net, if it happens

@AlexeyAkhunov AlexeyAkhunov marked this pull request as ready for review July 22, 2021 15:05
@AlexeyAkhunov AlexeyAkhunov merged commit 026c4e9 into devel Jul 22, 2021
@AlexeyAkhunov AlexeyAkhunov deleted the reorg-ropsten branch July 22, 2021 15:49
mandrigin pushed a commit that referenced this pull request Jul 22, 2021
* Reorg ropsten solution

* Add option

* Print TD recalc

* Correct fix for eip1559

* Try to fix the unwind

* Print header progress and hash

* Not insert descendants of bad headers

* Print some more

* Print less

* Better way of marking bad headers

* Disable inSync

* Penalise peers who give incorrect chain pieces

* better fix for initial cycle

* Clean up

Co-authored-by: Alexey Sharp <alexeysharp@Alexeys-iMac.local>
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 this pull request may close these issues.

2 participants