Skip to content

lightningd: --message-padding to allow users to disable padding altogether#9068

Merged
sangbida merged 1 commit intoElementsProject:masterfrom
rustyrussell:disable-message-padding
Apr 20, 2026
Merged

lightningd: --message-padding to allow users to disable padding altogether#9068
sangbida merged 1 commit intoElementsProject:masterfrom
rustyrussell:disable-message-padding

Conversation

@rustyrussell
Copy link
Copy Markdown
Contributor

In case there are more cases we missed, this lets users disable it.

Changelog-Added: Config: message-padding option can be set to false to disable it for all peers.

@rustyrussell rustyrussell added this to the v26.04 milestone Apr 19, 2026
…ogether.

In case there are more cases we missed, this lets users disable it.

Changelog-Added: Config: `message-padding` option can be set to `false` to disable it for all peers.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@rustyrussell rustyrussell force-pushed the disable-message-padding branch from 28d136d to 6ac1a1f Compare April 19, 2026 07:54

* **message-padding**=*BOOL*

Normally `connectd` will send extra bytes to peers to make messages
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

tiny nit: indentation

Comment thread CHANGELOG.md
### Added

- Protocol: we now pad all peer messages to make them the same length (excluding LND < v21 and current Eclair). ([#8893], [#9022])
- Config: `message-padding` option can be set to `false` to disable it for all peers. ([#9068])
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for this!

@sangbida sangbida merged commit deddd8e into ElementsProject:master Apr 20, 2026
83 of 87 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.

2 participants