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

Electrs not indexing unconfirmed transactions as spent #27

Open
softsimon opened this issue Jul 28, 2023 · 2 comments
Open

Electrs not indexing unconfirmed transactions as spent #27

softsimon opened this issue Jul 28, 2023 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@softsimon
Copy link
Member

softsimon commented Jul 28, 2023

We're seeing this on multiple (but not all) servers in production.

A transaction in the mempool https://node205.tk7.mempool.space/api/tx/b60793ec024f9425d8d7841d592adc8782b3d32921d0ce5a7e894bd62e00973c spending an input f7f8befc733c643f7455744d82e665a9c7dcc6fb37f1c0aa56862340aa0e698b is not registered in the address transactions API and shows up as unspent in the Outspends API endpoint https://node205.tk7.mempool.space/api/tx/f7f8befc733c643f7455744d82e665a9c7dcc6fb37f1c0aa56862340aa0e698b/outspend/344

{"spent":false}

Version mempool-electrs 3.0.0-dev-3c66022

On another server with the exact same githash, it does work https://node206.tk7.mempool.space/api/tx/f7f8befc733c643f7455744d82e665a9c7dcc6fb37f1c0aa56862340aa0e698b/outspend/344

{"spent":true,"txid":"b60793ec024f9425d8d7841d592adc8782b3d32921d0ce5a7e894bd62e00973c","vin":0,"status":{"confirmed":false}}

@softsimon softsimon added the bug Something isn't working label Jul 28, 2023
@junderw
Copy link
Sponsor Member

junderw commented Jul 28, 2023

Subtitle:
P2P bugs are fun.

If the git hash is the same, the only difference would be the order in which the unconfirmed transactions arrived. Confirmed transactions probably won't have any issues.

Trying to consolidate a bunch of mempools into one global mempool state is hard stuff.

I think #25 fixes this issue... But we need further testing.

@junderw
Copy link
Sponsor Member

junderw commented Aug 22, 2023

It's been about a month since #25 was merged.... has this issue re-appeared at all?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants
@softsimon @junderw @mononaut and others