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

[recovery]: try to CPFP anchor for restored channel #8280

Open
guggero opened this issue Dec 15, 2023 · 0 comments
Open

[recovery]: try to CPFP anchor for restored channel #8280

guggero opened this issue Dec 15, 2023 · 0 comments
Labels
anchors channel closing Related to the closing of channels cooperatively and uncooperatively P1 MUST be fixed or reviewed recovery Related to the backup/restoration of LND data (e.g. wallet seeds) SCB Related to static channel backup

Comments

@guggero
Copy link
Collaborator

guggero commented Dec 15, 2023

Background

Currently if a channel is restored using SCB we don't look at the mempool to figure out if there already is a commitment transaction published by the remote side, lnd just sits there and waits for a commitment transaction to confirm.
This assumes that the remote party will eventually CPFP their anchor to get the transaction confirmed.
But if the channel doesn't have any balance on the remote side, then they can't do it and the commitment TX will be stuck in the mempool if it doesn't have enough fees.

There is a workaround available now with chantools pullanchor (PR here: lightninglabs/chantools#100), but it would be great if we could inspect the mempool and attempt to sweep our anchor even if we restore a channel from SCB.

Steps to reproduce

  1. Open anchor channel, don't send any payments
  2. Reset funder node, restore channel from SCB
  3. Observe peer force closing but not pulling the anchor because they have no balance
  4. Channel just sits there
@guggero guggero added channel closing Related to the closing of channels cooperatively and uncooperatively recovery Related to the backup/restoration of LND data (e.g. wallet seeds) SCB Related to static channel backup anchors labels Dec 15, 2023
@saubyk saubyk added P2 should be fixed if one has time P1 MUST be fixed or reviewed and removed P2 should be fixed if one has time labels Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
anchors channel closing Related to the closing of channels cooperatively and uncooperatively P1 MUST be fixed or reviewed recovery Related to the backup/restoration of LND data (e.g. wallet seeds) SCB Related to static channel backup
Projects
None yet
Development

No branches or pull requests

2 participants