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

Add Splicing messages #2542

Closed
optout21 opened this issue Aug 30, 2023 · 1 comment · Fixed by #2544
Closed

Add Splicing messages #2542

optout21 opened this issue Aug 30, 2023 · 1 comment · Fixed by #2544

Comments

@optout21
Copy link
Contributor

optout21 commented Aug 30, 2023

As part of Splicing, add needed protocol messages.

The messages are defined in the LN spec PR 863 (draft).
Note that the PR is currently a draft, it needs a rebase, and does not reflect the latest state of discussions.

The following messages should be added:

  • splice (74)
  • splice_ack (76)
  • splice_locked (77)

The interactive TX negotiation messages (also used by Splicing) are already present. They are a good example for this task (PR #1794 ).
Note that the stfu quiescence message may be out of scope here currently.

@TheBlueMatt
Copy link
Collaborator

Like we did for channel-open-v2 I don't see anything wrong with landing this before we handle them, as long as we have some level of confidence that the spec isn't gonna change much going forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants