Skip to content

Conversation

@elnosh
Copy link

@elnosh elnosh commented Oct 30, 2025

Updates the naming from endorsement to accountability to match the latest state of the proposal to mitigate channel jamming attacks.

Removed the `experiment_start/end` dates. This is done
because in the previous iteration with endorsement
signaling, the recommendation was for the sender to
set it to 1 and that could have had privacy concerns
when first deploying given that the default was to
downgrade the signal to 0. Now the recommended default
for both sending and forwarding nodes is to set
`accountable` to 0. Hence, the dates have been
removed as nodes can start setting the signal
without leaking information about being the sender.
Copy link
Contributor

@carlaKC carlaKC left a comment

Choose a reason for hiding this comment

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

LGTM - just a pre-existing issue with one of the markdown links.

Working on a LDK implementation which we can link as reference to fill in the outstanding TODO.

cc: @thomash-acinq

| 256/257 | `hosted_channels` | This node accepts requests for hosted channels | IN | | [bLIP 17](./blip-0017.md) |
| 258/259 | `dns_resolver` | This node accepts DNSSEC proof requests | N | | [bLIP 32](./blip-0032.md) |
| 260/261 | `htlc_endorsement` | This node forwards experimental htlc endorsement signals | N | | [bLIP 4](./blip-004.md) |
| 260/261 | `htlc_accountability` | This node forwards experimental htlc accountability signals| N | | [bLIP 4](./blip-004.md) |
Copy link
Contributor

Choose a reason for hiding this comment

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

pre-existing: link should be to ./blip-0004.md, might as well fix while we're here!

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