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

asset wallet: introduce coin locking and coin selection mutex #261

Closed
guggero opened this issue Feb 9, 2023 · 2 comments · Fixed by #431
Closed

asset wallet: introduce coin locking and coin selection mutex #261

guggero opened this issue Feb 9, 2023 · 2 comments · Fixed by #431
Assignees

Comments

@guggero
Copy link
Member

guggero commented Feb 9, 2023

The asset wallet introduced in #244 does not allow a coin to be marked as being used (=locking), which needs to be done at the database layer.
Once we can mark asset coins as being used/locked, we also need to make sure that access to coin selection and the locking APIs are covered by a single coin selection mutex similar to how it's done in lnd.

@Roasbeef
Copy link
Member

May also want to have a bit that says if an output has been fully spent or not. Either that, or we need to delete it.

@dstadulis
Copy link
Collaborator

related issue #374

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging a pull request may close this issue.

3 participants