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.

Sign up for GitHub

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

Jump to bottom

Add a message ID to most of the to-device contents #1266

Closed
poljar opened this issue Dec 8, 2022 · 1 comment · Fixed by #1946
Closed

Add a message ID to most of the to-device contents #1266

poljar opened this issue Dec 8, 2022 · 1 comment · Fixed by #1946
Assignees
@poljar
Labels
encryption good first issue Good for newcomers
Milestone
Stabilize the cry...

Comments

@poljar
Copy link
Contributor

poljar commented Dec 8, 2022 •
edited by dkasak
Loading

To-device events are hard to track over federation, see element-hq/element-meta#558 for more info.

We should do something similar as matrix-org/matrix-js-sdk#2938 for the rust-sdk. The m.room_key events are the most important ones that should receive such a message ID.
@poljar poljar added good first issue Good for newcomers encryption labels Dec 8, 2022
@poljar poljar added this to the Stabilize the crypto crate milestone Dec 8, 2022
@ara4n
Copy link
Member

ara4n commented Dec 11, 2022

This is fairly urgent, presumably, given ER & EX now flying around in the field, and the fact that EI/EA/EW all have it already?

@poljar poljar mentioned this issue Jan 13, 2023
Merged
@poljar poljar self-assigned this Jan 13, 2023
@poljar poljar mentioned this issue May 19, 2023
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@poljar poljar

Labels
encryption good first issue Good for newcomers
Projects
None yet
Milestone
Stabilize the crypto crate
Development

Successfully merging a pull request may close this issue.

Message ID support: The sending side matrix-org/matrix-rust-sdk
2 participants
@poljar @ara4n