-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for key backups #22
Conversation
74f8864
to
e885b88
Compare
This currently merges in a few dependent branches that should be submitted before this PR is. |
on account of it being removed from the crypto-sdk
OlmMachine.receiveSyncChanges returns an array of [device messages, room key changes], so emit "to_device.decrypted" with that instead of the entire array. Fixes regression introduced by turt2live#287.
Also condense the validity checks on the returned value
834e539
to
64463f4
Compare
Also add/tweak some utility types to help with this
The added test covers creating & retrieving key backup versions (i.e. which pubkey to use, and what backup version it is). Yet to be tested is the act of backing up keys. |
The remaining tests have been added, and it was very much worth it, as doing so revealed a bug & a few code improvements to be made. Barring review feedback, this PR should be good to go. |
Checklist
[ ] Sign-off given on the changes (see CONTRIBUTING.md)