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

Own change output from funding tx is not shown in the balance #2286

Open
holzeis opened this issue Mar 20, 2024 · 3 comments
Open

Own change output from funding tx is not shown in the balance #2286

holzeis opened this issue Mar 20, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@holzeis
Copy link
Contributor

holzeis commented Mar 20, 2024

Labeling this as bug as it used to behave differently before.

@holzeis holzeis added the bug Something isn't working label Mar 20, 2024
@luckysori
Copy link
Contributor

@holzeis: Do you mean that the change output is missing when the transaction is still unconfirmed? I can reproduce that, but once the transaction is confirmed, the balance is updated correctly.

We should still include pending outputs in the wallet balance somehow.

@holzeis
Copy link
Contributor Author

holzeis commented Mar 25, 2024

@holzeis: Do you mean that the change output is missing when the transaction is still unconfirmed?

Yes, we used to trust unconfirmed change output from our own transactions.

@luckysori
Copy link
Contributor

I think this can be fixed by adding a get_change_address method to rust-dlc's Wallet trait.

At the moment, there is only a get_new_address method which is used for every situation. bdk can trust unconfirmed payments to change addresses, but not unconfirmed payments to external addresses.

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

2 participants