fix: delete vss network graph using new ffi client#775
Merged
Conversation
2 tasks
82a5527 to
ba44df3
Compare
ba44df3 to
f120581
Compare
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
3dfa34b to
e556b04
Compare
This comment has been minimized.
This comment has been minimized.
Member
jvsena42
reviewed
Feb 11, 2026
jvsena42
reviewed
Feb 11, 2026
jvsena42
reviewed
Feb 11, 2026
Member
|
LGTM, testing ... |
This comment was marked as resolved.
This comment was marked as resolved.
Collaborator
Author
|
drafted to update to new vss-rust-client which separates the app client from the vss client |
# Conflicts: # app/src/main/java/to/bitkit/services/LightningService.kt
jvsena42
reviewed
Feb 13, 2026
jvsena42
approved these changes
Feb 13, 2026
Member
jvsena42
left a comment
There was a problem hiding this comment.
Performed Lightning operation on main net
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
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.
Closes #765
This PR:
Description
Builds on #740. The stale graph auto-recovery added in #765 attempted to deleted the graph from VSS using the app-level backup client but due to different key obfuscation namespacing in ldk-node, the graph backup was never deleted.
This PR switches to a dedicated LDK-specific FFI client implementation that points operations to the correct obfuscated file names backed up by ldk-node on VSS. This new VSS client implementation adjusted for supporting OPs on ldk-node backups is found in PR:
In practice this translate to successful LN payments to nodes that would otherwise fail, one example being payments to Blink wallet if having a stale graph in your wallet's VSS backups.
LDK-node already has the required mechanism to trigger redownload of the network graph if attempts to retrieve it from its backup on VSS fails. The cleanup mechanism is triggered when the graph is found to miss any of the expected trusted peers, which in turn triggers ldk-node to kick-off its re-download of the graph from the latest RGS server snapshot.
Note
After more research this issue proved to reproduce only on wallets created with the early release candidate versions of the app, when caching of network graph was still done on VSS.
Preview
Test 1️⃣ - Payment to Blink
pay2blink.mp4
Test 2️⃣ - VSS app data preserved across
vss-rust-clientupdate0.4.0)0.5.5)QA Notes
Tip
No need to go all the way to replicate a stale graph on your VSS backup. To validate, simply test on mainnet some random LN payments to a couple wallets you have on other apps like Blink, Phoenix, WoS. If payments succeed, PR is good to be merged.
1. Verify stale graph reset clears VSS via FFI client
Network graph is stale, resetting and restarting...Cleared stale network graph from VSS (first delete)2. Verify VSS app data preserved across versions
master)