Skip to content

Autosquash splice fixup commits#8949

Merged
sangbida merged 4 commits intoElementsProject:masterfrom
sangbida:master
Mar 19, 2026
Merged

Autosquash splice fixup commits#8949
sangbida merged 4 commits intoElementsProject:masterfrom
sangbida:master

Conversation

@sangbida
Copy link
Copy Markdown
Collaborator

Just rewriting git history to remove some fixup commits!

Changelog-None

ddustin and others added 4 commits March 19, 2026 10:17
In some situations Eclair sends CHANNEL_READY before resuming the splice in reestablish. This update allows one of these messages to be processed if it comes in during the splice resume process.

Changelog-None
Splice commands can fail because of external reasons, for example our peer rejecting the splice. These external reasons eat the active `splice_command` so we need to stop storing the `splice_command` in callbacks and instead pull it from the `splice_commands` list which automatically removes the object when its consumed.

This corrects a lightningd crash under certain RPC circumstances.

Changelog-None
A new TLV in `channel_reestablish` lets our peer request announcement signatures. This updates CLN to (re)send them when requested.

As per splice spec PR https://github.com/lightning/bolts/pull/1160/changes#diff-ed04ca2c673fd6aabde69389511fa9ee60cb44d6b2ef6c88b549ffaa753d6afeR3303-R3308

Changelog-None
If the execution of reckless fails for some reason recklessrpc plugin
should return an rpc failure message instead of hanging.

Changelog-Fixed: JSON-RPC: reckless command no longer hangs if reckless executable is not found in PATH.

Signed-off-by: Lagrang3 <lagrang3@protonmail.com>
Copy link
Copy Markdown
Collaborator

@ShahanaFarooqui ShahanaFarooqui left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 8c8364b

@sangbida sangbida merged commit daa994f into ElementsProject:master Mar 19, 2026
8 checks passed
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 this pull request may close these issues.

4 participants