Skip to content

Conversation

@ovitrif
Copy link
Collaborator

@ovitrif ovitrif commented Aug 19, 2025

Related:

This PR integrates with vss-rust-client-ffi v0.2.0 to enable encrypted data backups with authenticated calls to vss-server.

Description

  • feat: vss-client lnurl-auth
  • chore: update ldk-node to v0.6.2-rc.1
  • chore: auto-format e2e-test-ids.md

QA Notes

Test using vss-server with jwt auth and lnurl-server in docker via bitkit-docker:

  • Run the bitkit-docker docker-compose setup including the vss-server fork with the JWT auth changes
  • Uncomment in Env.kt the localhost URLs for vssServerUrl & lnurlAuthSeverUrl
  • Use a wallet with LN funds, add a widgets + remove suggestions until only one card remains
  • Uninstall app (wiping its storage)
  • Reinstall and restore wallet from seed
  • Expect LN channels + widgets & suggestion cards backup restored
  • Reset suggestions, remove widget
  • Check in logcat that all vss data backup logs succeed — multiple backup successes validate the versioning changes

@ovitrif ovitrif self-assigned this Aug 19, 2025
@ovitrif ovitrif marked this pull request as ready for review August 20, 2025 09:54
@ovitrif ovitrif force-pushed the feat/vss-client-auth branch from 9dbbecc to 7c5998b Compare August 20, 2025 10:02
@ovitrif ovitrif requested a review from jvsena42 August 20, 2025 10:04
jvsena42
jvsena42 previously approved these changes Aug 20, 2025
@ovitrif
Copy link
Collaborator Author

ovitrif commented Aug 20, 2025

@jvsena42 Added a small change after the vss-server with auth was deployed, a commented line of code to make it easy to switch to the remote vss-server for testing purposes.

Needing another review, then we can merge this.

@ovitrif ovitrif requested a review from jvsena42 August 20, 2025 11:48
@jvsena42 jvsena42 merged commit 7301a2f into master Aug 20, 2025
5 checks passed
@jvsena42 jvsena42 deleted the feat/vss-client-auth branch August 20, 2025 12:20
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