Skip to content

Cache LSPS2 get_info responses #977

Open
benthecarman wants to merge 2 commits into
lightningdevkit:mainfrom
benthecarman:cache-lsps2
Open

Cache LSPS2 get_info responses #977
benthecarman wants to merge 2 commits into
lightningdevkit:mainfrom
benthecarman:cache-lsps2

Conversation

@benthecarman

Copy link
Copy Markdown
Contributor

Reuse opening fee offers while they remain valid and match the configured LSP token. This avoids repeated LSPS2 round trips across node restarts while retaining a fresh-response fallback.

Evict cached offers when a buy request fails so a retry fetches updated parameters.

Reuse opening fee offers while they remain valid and match the
configured LSP token. This avoids repeated LSPS2 round trips across
node restarts while retaining a fresh-response fallback.

Evict cached offers when a buy request fails so a retry fetches
updated parameters.

This commit was created with assistance from OpenAI Codex.
Read cached fee offers before connecting so cache hits do not open
connections to every configured LSP. Connect only for cache misses and
ensure the selected LSP is connected before sending a buy request.

This commit was created with assistance from OpenAI Codex.
@ldk-reviews-bot

ldk-reviews-bot commented Jul 13, 2026

Copy link
Copy Markdown

I've assigned @tnull as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

@ldk-reviews-bot ldk-reviews-bot requested a review from tnull July 13, 2026 18:56
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