Skip to content

Check running state at public API boundaries - #1169

Open
tnull wants to merge 1 commit into
lightningdevkit:mainfrom
tnull:2026-10-outer-api-running-checks
Open

tnull wants to merge 1 commit into
lightningdevkit:mainfrom
tnull:2026-10-outer-api-running-checks

Conversation

@tnull

@tnull tnull commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator

Just a small follow-up to #1160.

Some live-node APIs still relied on helper methods to reject calls while stopped, leaving lifecycle validation inconsistent across the public API.

Reject stopped-node calls at each public entry point and remove the helper checks. Keep unified payment rechecks before dispatch so a stop during asynchronous resolution is still detected.

@ldk-reviews-bot

ldk-reviews-bot commented Oct 15, 2026 •

Copy link
Copy Markdown

👋 I see @TheBlueMatt was un-assigned.
If you'd like another reviewer assignment, please click here.

@tnull
tnull requested review from tankyleo and removed request for TheBlueMatt October 15, 2026 11:25
@tnull

tnull commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator Author

Reassigning @tankyleo.

Comment thread src/payment/spontaneous.rs
Some live-node APIs still relied on helper methods to reject calls
while stopped, leaving lifecycle validation inconsistent across the
public API.

Reject stopped-node calls at each public entry point and remove the
helper checks. Keep unified payment rechecks before dispatch so a stop
during asynchronous resolution is still detected. Cover the payment
variants with offline assertions that stopped calls return NotRunning.

Co-Authored-By: HAL 9000
@tnull
tnull force-pushed the 2026-10-outer-api-running-checks branch from 8b76e56 to a3a1c59 Compare October 16, 2026 13:36
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.

3 participants