Skip to content

[WIP] Encrypted backup#1807

Draft
pythcoiner wants to merge 11 commits intowizardsardine:masterfrom
pythcoiner:encrypted_backup
Draft

[WIP] Encrypted backup#1807
pythcoiner wants to merge 11 commits intowizardsardine:masterfrom
pythcoiner:encrypted_backup

Conversation

@pythcoiner
Copy link
Collaborator

@pythcoiner pythcoiner commented Aug 8, 2025

  • Installer: import encrypted descriptor
  • Installer: export encrypted descriptor
  • Installer: import encrypted backup
  • Settings: export encrypted descriptor
  • Settings: export encrypted backup
  • Settings: import encrypted backup
  • Connect: import encrypted descriptor/backup

Closes #1803 & #1804

image image image image

decryption can be tested by using the cli of encrypted_descriptor on liana branch.
you can build the cli with

cargo build --release --features "cli miniscript_latest"

@pythcoiner pythcoiner marked this pull request as draft August 8, 2025 08:06
@pythcoiner pythcoiner force-pushed the encrypted_backup branch 3 times, most recently from 899c116 to dc9b3b9 Compare August 8, 2025 14:23
@nondiremanuel nondiremanuel moved this to In Progress in Liana General Aug 8, 2025
@pythcoiner pythcoiner force-pushed the encrypted_backup branch 2 times, most recently from 4b5d6fb to 75920c5 Compare August 27, 2025 09:30
@pythcoiner pythcoiner changed the title [WIP] Encrypted backup export [WIP] Encrypted backup Aug 27, 2025
@pythcoiner pythcoiner force-pushed the encrypted_backup branch 6 times, most recently from eddf492 to 9c91a30 Compare September 1, 2025 11:12
@pythcoiner pythcoiner force-pushed the encrypted_backup branch 4 times, most recently from ca68b1f to ef24341 Compare September 4, 2025 09:22
@pythcoiner pythcoiner marked this pull request as ready for review September 4, 2025 09:23
@pythcoiner pythcoiner marked this pull request as draft September 8, 2025 08:00
@pythcoiner pythcoiner mentioned this pull request Sep 8, 2025
@pythcoiner
Copy link
Collaborator Author

Moving this to draft as we decided to release only encrypted descriptor in v13 in order to address some suggestions on the draft BIP

see #1846 for what is expected to be ship in v13

edouardparis added a commit that referenced this pull request Sep 18, 2025
bbe75f7 export: add test for decrypt descriptor from file (pythcoiner)
5e27112 decrypt: move to installer (pythcoiner)
b278ef2 export: sanitize encrypted backup before writing to disk (pythcoiner)
438820e liana-sdk: implement LianaDescriptor.spendable_keys()/.process_unspendable_key() (pythcoiner)
4c4fc03 installer: impl import encrypted backup/descriptor for liana-connect (pythcoiner)
b3788d5 installer: move ImportDescriptorModal with some logic to import_descriptor.rs (pythcoiner)
b68cdff export: remove unused variant of ImportExportType (pythcoiner)
c754316 installer: implement import encrypted descriptor (pythcoiner)
d700719 gui: wording + add encrypted descriptor export (pythcoiner)
f728377 installer: export encrypted descriptor instead plaintext backup (pythcoiner)
d5e8aa5 liana-gui: move default_derivation_path() to utils (pythcoiner)
b866bfe gui: add dependency for encrypted backup (pythcoiner)
298615d liana-sdk: add a getter for LianaDescriptor.multi_desc (pythcoiner)

Pull request description:

  This is a lightweight version of #1807, where we do not encrypt for a backup export.
  (I just dropped the commit that enable encryption on backup export)

ACKs for top commit:
  jp1ac4:
    ACK bbe75f7.

Tree-SHA512: 4b58654432bb2158ca0e15f4012f5b8e04864bfd61aa9d61b28c0ce56b822053b2a7e62b626b807d8a76d0fc5549bbab26858a7ecd6862c433f7dc52c91b3dec
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Allow the user to back up an encrypted version of the descriptor

3 participants