Fix websocket issues and mint quotes #1246

Merged
crodas merged 2 commits from fix/websocket-bolt11-check-prev-state into main 2025-11-03 00:06:03 +00:00
crodas commented 2025-11-02 01:38:02 +00:00 (Migrated from github.com)

Description

Fixes #1209 and #1104

When checking with HTTP pulling, mint quotes would ping the ln backend if needed to get any payment that may not be included, so the mint uses this chance to correct its database.

This check was missing for database subscriptions

Notes to the reviewers


Suggested CHANGELOG Updates

CHANGED

ADDED

REMOVED

FIXED


Checklist

### Description Fixes #1209 and #1104 When checking with HTTP pulling, mint quotes would ping the ln backend if needed to get any payment that may not be included, so the mint uses this chance to correct its database. This check was missing for database subscriptions ----- ### Notes to the reviewers <!-- In this section you can include notes directed to the reviewers, like explaining why some parts of the PR were done in a specific way --> ----- ### Suggested [CHANGELOG](https://github.com/cashubtc/cdk/blob/main/CHANGELOG.md) Updates <!-- Please do not edit the actual changelog but note what you changed here. --> #### CHANGED #### ADDED #### REMOVED #### FIXED ---- ### Checklist * [ ] I followed the [code style guidelines](https://github.com/cashubtc/cdk/blob/main/CODE_STYLE.md) * [ ] I ran `just final-check` before committing
thesimplekid (Migrated from github.com) reviewed 2025-11-02 22:11:09 +00:00
thesimplekid (Migrated from github.com) left a comment

Nice looks good just think one doc comment needs to be fixed.

Nice looks good just think one doc comment needs to be fixed.
@ -1,11 +1,14 @@
//! Specific Subscription for the cdk crate
thesimplekid (Migrated from github.com) commented 2025-11-02 22:06:36 +00:00

I think this comment is cut off?

I think this comment is cut off?
crodas (Migrated from github.com) reviewed 2025-11-02 22:26:20 +00:00
@ -1,11 +1,14 @@
//! Specific Subscription for the cdk crate
crodas (Migrated from github.com) commented 2025-11-02 22:26:20 +00:00

Yeah, that was the old doc-doc from a previous approach. Lemme update it.

Yeah, that was the old doc-doc from a previous approach. Lemme update it.
thesimplekid (Migrated from github.com) approved these changes 2025-11-03 00:05:45 +00:00
cdk-bot commented 2025-11-03 00:06:42 +00:00 (Migrated from github.com)

Backport failed for v0.13.x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin v0.13.x
git worktree add -d .worktree/backport-1246-to-v0.13.x origin/v0.13.x
cd .worktree/backport-1246-to-v0.13.x
git switch --create backport-1246-to-v0.13.x
git cherry-pick -x c6434b88d1498e66b3a43619612f836a2b383ada
Backport failed for `v0.13.x`, because it was unable to cherry-pick the commit(s). Please cherry-pick the changes locally and resolve any conflicts. ```bash git fetch origin v0.13.x git worktree add -d .worktree/backport-1246-to-v0.13.x origin/v0.13.x cd .worktree/backport-1246-to-v0.13.x git switch --create backport-1246-to-v0.13.x git cherry-pick -x c6434b88d1498e66b3a43619612f836a2b383ada ```
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
cashubtc/cdk!1246
No description provided.