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

Cancel back outgoing dust htlcs before commitment is confirmed. #9068

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Commits on Oct 17, 2024

  1. Configuration menu
    Copy the full SHA
    69c2435 View commit details
    Browse the repository at this point in the history
  2. contratcourt: refactor resolving htlc logic

    Refactor the part where we are failing back the incoming htlc
    when the channel of the corresponding outgoing htlc is force
    closed.
    ziggie1984 committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    2d7fd28 View commit details
    Browse the repository at this point in the history
  3. contractcourt: enhance chainAction type

    We distinguish between dangling and dust htlcs. This does not
    change any logic but only introduces new types to later act on them
    differently when we begin to fail dust htlcs earlier in a later
    commit.
    ziggie1984 committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    23b3516 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    c97960a View commit details
    Browse the repository at this point in the history
  5. contractcourt: Cancel dust htlcs prematurely

    We will now cancel dust htlcs on the local/remote commits after
    we decided to go onchain. This can be done because dust cannot
    be enforced onchain and therefore there is no way to also reveil
    the preimage onchain.
    ziggie1984 committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    f4a85b6 View commit details
    Browse the repository at this point in the history
  6. contratcourt: dont consider dust htlc for anchor sweep

    Now that we cancel dust htlcs prematurely even before the
    commitment tx is confirmed we don't consider dust htlcs when
    creating the cpfp transaction.
    ziggie1984 committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    3109694 View commit details
    Browse the repository at this point in the history
  7. itest: adopt itests for the new behavior

    Now outgoing dust-htlcs are canceled back before the commitment
    is confirmed onchain.
    ziggie1984 committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    7fb61e6 View commit details
    Browse the repository at this point in the history
  8. Configuration menu
    Copy the full SHA
    e25afc2 View commit details
    Browse the repository at this point in the history