-
Notifications
You must be signed in to change notification settings - Fork 286
crypto: encrypt, upload and share keys for room history #4864
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
Conversation
41d75d1
to
ea0b176
Compare
f93d4a6
to
3149616
Compare
211b9ad
to
7cb525f
Compare
632e1b6
to
e963311
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4864 +/- ##
==========================================
+ Coverage 85.84% 85.95% +0.11%
==========================================
Files 323 324 +1
Lines 35515 35611 +96
==========================================
+ Hits 30487 30610 +123
+ Misses 5028 5001 -27 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
6963c22
to
548cd49
Compare
I wish github would show when someone presses the "Start Review" button, I guess I could have also told you. Is there more coming, should I wait for things to calm down? |
oh, sorry. Occasionally we use an 👀 reaction on the PR to denote "I'm looking at this". I'm not planning any more work on this, though that was also the case this morning, and then things changed. Let me see what's wrong with the CI, then I'll leave it alone. |
5b6962b
to
000b630
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left some comments and questions. I think most of it makes sense.
crates/matrix-sdk-crypto/src/session_manager/group_sessions/mod.rs
Outdated
Show resolved
Hide resolved
crates/matrix-sdk-crypto/src/session_manager/group_sessions/mod.rs
Outdated
Show resolved
Hide resolved
crates/matrix-sdk-crypto/src/session_manager/group_sessions/mod.rs
Outdated
Show resolved
Hide resolved
7ba8967
to
2420934
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, looks good now.
The next step in our work on sharing encrypted room history. Add a method to `matrix_sdk::room::Room` which will upload an encrypted key bundle.
For each device belonging to the user, encrypt and send to-device messages containing the bundle data
This is only a partial test, since we haven't yet implemented the receiver side of the history-sharing messages.
a600c49
to
80db096
Compare
Signed-off-by: Richard van der Hoff <[email protected]>
Signed-off-by: Richard van der Hoff <[email protected]>
The next step of our plan to share room history (element-hq/element-meta#39). This PR adds the functionality to:
Fixes: #4505