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

feat: detect multiple UTXOs per output #41

Merged
merged 1 commit into from
Jan 7, 2019

Conversation

michael1011
Copy link
Member

This PR refactors the way UTXOs and addresses are handled by separating the UTXOs and outputs (scripts of the addresses) in the database which means that:

  • every address can have multiple UTXOs in an efficient way (also known as address reuse; should be avoided if possible)
  • there are no problems if a single transaction sends funds to multiple addresses of Boltz (the transaction hash used to be the primary key of the utxos table in the database)
  • Boltz listens new transactions on all addresses the wallet ever created. Before this PR Boltz forgot about old addresses it created on every restart.

This is a breaking change to the database. Please remove your existing boltz.db before running Boltz.

@michael1011 michael1011 merged commit 9b3c709 into master Jan 7, 2019
@ghost ghost removed the in progress label Jan 7, 2019
@michael1011 michael1011 deleted the multiple-utxo-same-output branch January 7, 2019 16:37
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.

None yet

2 participants