chore: bump HTLC expiry threshold to 45s#5
Merged
martinsaposnic merged 1 commit intolsp-0.2.0from Mar 3, 2026
Merged
Conversation
The 10s threshold was too tight for the webhook delivery pipeline, causing premature HTLC expiry and RetriesExhausted failures on 402 payments. 45s gives ample room for webhook round-trips while still well within CLTV delta bounds.
martinsaposnic
added a commit
to moneydevkit/ldk-node
that referenced
this pull request
Mar 3, 2026
Fire HTTP POST directly in handle_next_event() instead of routing through the event queue (S3 persistence), cutting webhook latency from 15-23s to <2s. LiquiditySource gets optional reqwest::Client + webhook_url fields. When configured, SendWebhook events bypass the event queue entirely. Falls back to event queue if not configured. Also bumps rust-lightning to include HTLC expiry threshold increase (10s -> 45s, moneydevkit/rust-lightning#5, 1432d063c).
4 tasks
martinsaposnic
added a commit
to moneydevkit/ldk-node
that referenced
this pull request
Mar 3, 2026
Fire HTTP POST directly in handle_next_event() instead of routing through the event queue (S3 persistence), cutting webhook latency from 15-23s to <2s. LiquiditySource gets optional reqwest::Client + webhook_url fields. When configured, SendWebhook events bypass the event queue entirely. Falls back to event queue if not configured. Also bumps rust-lightning to include HTLC expiry threshold increase (10s -> 45s, moneydevkit/rust-lightning#5, 1432d063c).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
HTLC_EXPIRY_THRESHOLD_SECSfrom 10 to 45 in LSPS4 serviceRetriesExhaustedon 402 paymentsTest plan