-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Labels
anchorsbugUnintended code behaviourUnintended code behaviourchannel closingRelated to the closing of channels cooperatively and uncooperativelyRelated to the closing of channels cooperatively and uncooperativelyforce closes
Milestone
Description
LND 18 has a new reworked sweeper subsystem. Before we would register the Anchors with the sweeper subsystem every time a channel was force closed. Right now we check if HTLCs are on the channel when it goes onchain. If it is not the case we do not register the anchors with our sweeper subsystem.
Now if the user wants to bump the commitment tx (lncli wallet bumpforceclosefee) it is not possible because no anchor sweep is registered. It just quietly fails on the lncli level.
Thought a bit about the design because I think this is a mandatory feature, commitment fees might be very low so using the anchor to accelerate the confirmation.
I came up with the following idea, happy to hear your thoughts before implementing:
- Register the anchor with the sweeper, but don't signal sweeping. So we have all the available data at hand when trying to bump the anchor channel.
- Transferring most of the logic from the lncli level to the rpc level so that we can also use it more independently.
saubyk and M1ch43lVsaubyk
Metadata
Metadata
Assignees
Labels
anchorsbugUnintended code behaviourUnintended code behaviourchannel closingRelated to the closing of channels cooperatively and uncooperativelyRelated to the closing of channels cooperatively and uncooperativelyforce closes