Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

WIP: online incremental megolm backups #1703

Closed
wants to merge 3 commits into from
Closed

Conversation

ara4n
Copy link
Member

@ara4n ara4n commented Jan 15, 2018

This doesn't remotely work yet (not even tried to compile it yet) and isn't finished, but opening a PR to aid feedback/review

Listens to new crypto.suggestKeyRestore event, and if it fires shows a new SuggestKeyRestoreDialog to prompt the user to restore their old keys; either by restoring from online backup, importing offline backup, or verifying with a device such that the device syncs their keys over. It also adds a SuggestKeyBackupDialog to prompt the user to backup their old keys.

Stuff left to do includes:

  • Encrypting and saving new inbound sessions to the server (i.e. turning the comments into code)
  • Implementing onRecoverFromBackup (turning comments into code)
  • Implementing onStartNewBackup (generate backup key; give it to the user; encrypt all the keys and upload them)
  • Hooking up the various checks on backup state to js-sdk (and thus Olm)
  • Hooking up online backups so you can enable/disable them from UserSettings

Later on:

  • Proactively sharing all sessions when you verify your own device
  • Cross-sign and publish on /devices the devices you've verified yourself, both so people can trust you (or distrust you if you revoke), and also so we know whether to prompt to restore keys or not!
  • Implementing better verification so users just have to compare a phrase (e.g. uhoreg's https://cryptpad.fr/pad/#/1/edit/0eqaXl2wZhiq5s5Ty-wzcg/QL4+6V9poZCmbZI0ispNMFbU/)

This is twinned with...

@dbkr
Copy link
Member

dbkr commented Sep 18, 2018

Superseded by #2169

@dbkr dbkr closed this Sep 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants