-
Notifications
You must be signed in to change notification settings - Fork 391
Esplora blockchain fails reorg test #357
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
Comments
Hello @LLFourn is this still available? |
Yep this is still a problem -- that line of code is still there. |
@gabbyprecious would you rather work on this one or #518? I'd rather new folks take one issue at a time. |
Hi, i came from summerofbitcoin and if this issue is still available i can try to fix it |
Hi @Gioelio do you still want to work on this issue or did you take another summer of bitcoin project? |
I tried to fix it but I didn't understand what the problem was, sorry |
I have some time today to work on this. I can tackle this one if no one else is already doing it. |
FYI I've done some research on how electrum reacts to reorgs and I think it could be related to why this and the electrum (with version 0.9.1) integration tests fail. See here: https://bitcoin.stackexchange.com/questions/114044/how-can-i-simulate-a-reorg-for-testing |
New |
see:
bdk/src/testutils/blockchain_tests.rs
Line 534 in 881ca8d
Close this issue by getting to the bottom of it and making it pass.
The text was updated successfully, but these errors were encountered: