Skip to content
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

Move to using Olm for to-device signalling #33

Closed
robertlong opened this issue Oct 4, 2021 · 3 comments
Closed

Move to using Olm for to-device signalling #33

robertlong opened this issue Oct 4, 2021 · 3 comments
Labels
p2 Should fix/implement, but not at the expense of p1s T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements

Comments

@robertlong
Copy link
Contributor

To-device signalling is currently sent via plaintext. We should switch to sending them with Olm

@robertlong robertlong added the T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements label Oct 4, 2021
@robertlong
Copy link
Contributor Author

I need more information on how to encrypt to device messages via OLM. What contents should be encrypted? Is there any other specifics to how this should be implemented?

@robertlong
Copy link
Contributor Author

@ara4n provided an example of this already in the matrix-js-sdk:
https://github.com/matrix-org/matrix-js-sdk/blob/develop/src/crypto/algorithms/megolm.ts#L1795-L1850

Moving this back to To do, but de-prioritizing while we work out some signaling bugs.

@robertlong robertlong added the p2 Should fix/implement, but not at the expense of p1s label Oct 16, 2021
@dbkr
Copy link
Member

dbkr commented May 18, 2022

Fixed by #303

@dbkr dbkr closed this as completed May 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2 Should fix/implement, but not at the expense of p1s T-Enhancement New features, changes in functionality, performance boosts, user-facing improvements
Projects
None yet
Development

No branches or pull requests

3 participants