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

Investigate electrs UTXO lookup #368

Closed
gregdhill opened this issue Sep 14, 2022 · 0 comments · Fixed by #463
Closed

Investigate electrs UTXO lookup #368

gregdhill opened this issue Sep 14, 2022 · 0 comments · Fixed by #463

Comments

@gregdhill
Copy link
Member

There is a problem with the current light client implementation (see this comment) that we may attempt to fund a transaction with a stale UTXO (reported by Electrs) if there is a parallel payment in the mempool. Firstly, we should investigate and note when exactly UTXOs are marked as spent by Electrs and then implement one of the following:

  1. Hold the transaction creation mutex until the block has been included and Electrs marks this as spent
  2. Cache UTXOs we are spending from and skip these when funding a new transaction
@gregdhill gregdhill added this to the Vault Light Client milestone Sep 14, 2022
@nud3l nud3l added this to Backlog Jan 3, 2023
@nud3l nud3l moved this to Todo ⏳ in Backlog Jan 3, 2023
@github-project-automation github-project-automation bot moved this from Todo ⏳ to Done ✅ in Backlog Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant