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

unexpected funds #676

Open
ubbabeck opened this issue Oct 14, 2023 · 4 comments
Open

unexpected funds #676

ubbabeck opened this issue Oct 14, 2023 · 4 comments
Assignees

Comments

@ubbabeck
Copy link
Contributor

When the internal wallet has a confirmed on-chain tx we display unxepected funds warning, but the wallet does not check if the funds are reserved until block height > now so when the user tries to spend the funds they receive an error.

Solution: show warning dialog with info about the reserved utxos if the blockheight isn't reached yet.

@kingonly
Copy link
Member

@ubbabeck what's the use case? What's Breez mobile behavior? Why are the funds reserved?

@roeierez
Copy link
Member

Funds are reserved in case the user has already called sweep. In that case core lightning reserves the funds.
I think a better course of action would be to unreserve the funds before broadcasting the sweep transaction (https://docs.corelightning.org/reference/lightning-unreserveinputs).
This way the user has an option to replace a low fee already broadcasted transaction with a new one.

@roeierez roeierez added this to the v0.2.4 alpha(sdk) milestone Oct 15, 2023
@ubbabeck ubbabeck self-assigned this Oct 16, 2023
@ubbabeck
Copy link
Contributor Author

Got it, thanks for clarifying @roeierez.

@ubbabeck
Copy link
Contributor Author

ubbabeck commented Oct 18, 2023

Created issue in breez-sdk breez/breez-sdk-greenlight#531.

@ubbabeck ubbabeck reopened this Nov 21, 2023
@ubbabeck ubbabeck removed this from the v0.2.4 alpha(sdk) milestone Dec 4, 2023
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

No branches or pull requests

3 participants