Skip to content

[feature]: add custom chan support to new rbf chan closer #9663

Description

@Roasbeef

Is your feature request related to a problem? Please describe.

Today the RBF chan closer doesn't observe/propagate the auxOutputs functional option like the legacy closing protocol does:

// Before we complete the cooperative close, we'll see if we
// have any extra aux options.
c.auxOutputs, err = c.auxCloseOutputs(remoteProposedFee)
if err != nil {
return noClosing, err
}
c.auxOutputs.WhenSome(func(outs AuxCloseOutputs) {
closeOpts = append(
closeOpts, lnwallet.WithExtraCloseOutputs(
outs.ExtraCloseOutputs,
),
)
closeOpts = append(
closeOpts, lnwallet.WithCustomCoopSort(
outs.CustomSort,
),
)
})

Describe the solution you'd like

We should update the state transition in the new RBF closer to support this feature (permits > 1 output to the sender on a co-op close transaction).

No activity

Activity on this issue will appear here.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

channel closingRelated to the closing of channels cooperatively and uncooperativelyenhancementImprovements to existing features / behaviourprotocolrbf

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions