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

Implement MSC3061: Sharing room keys for past messages on Invite #39

Open
4 of 9 tasks
Tracked by #18
dkasak opened this issue Oct 16, 2021 · 9 comments
Open
4 of 9 tasks
Tracked by #18

Implement MSC3061: Sharing room keys for past messages on Invite #39

dkasak opened this issue Oct 16, 2021 · 9 comments
Assignees
Labels
App: Android App: iOS O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience T-Enhancement T-Epic Issue is at Epic level Team: Crypto Z-Chronic

Comments

@dkasak
Copy link
Member

dkasak commented Oct 16, 2021

@turt2live turt2live changed the title MSC3061: Sharing room keys for past messages Implement MSC3061: Sharing room keys for past messages Oct 16, 2021
@BillCarsonFr
Copy link
Member

BillCarsonFr commented Apr 25, 2022

SCOPE UPDATE

There are some scalability issues with sending the keys for the full history, as well as several edge cases. We don't want to fix all of them on existing SDKs (mobile, web).
The Rust SDK implementation should be complete.

So for existing SDKS the main goal is to give the new joiner the immediate context on the room to know why he was invited, so basically the recent histoty not full history.

@BillCarsonFr BillCarsonFr added the T-Epic Issue is at Epic level label Oct 13, 2022
@BillCarsonFr BillCarsonFr added this to the Crypto Q4 milestone Oct 13, 2022
@BillCarsonFr BillCarsonFr changed the title Implement MSC3061: Sharing room keys for past messages Implement MSC3061: Sharing room keys for past messages on Invite Oct 13, 2022
@BillCarsonFr BillCarsonFr added Z-Chronic and removed T-Epic Issue is at Epic level labels Oct 13, 2022
@kittykat kittykat added T-Epic Issue is at Epic level App: Android App: iOS T-Enhancement O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience labels Oct 20, 2022
@benparsons
Copy link
Member

Do we think implementation of MSC3061 should include web? element-hq/element-web#22999 is a web UI task for room history key sharing that is currently biting a customer fairly badly. If we could include this task here it would be helpful.

@richvdh
Copy link
Member

richvdh commented Nov 16, 2022

Do we think implementation of MSC3061 should include web? vector-im/element-web#22999 is a web UI task for room history key sharing that is currently biting a customer fairly badly. If we could include this task here it would be helpful.

I'm reasonably sure this is already implemented on web. AFAICT element-hq/element-web#22999 is just a bug, not a missing implementation.

EDIT 2024-06-13: this is no longer true, because the Rust crypto stack doesn't implement it; tracked at element-hq/element-web#26867

@thoraj
Copy link

thoraj commented Sep 5, 2024

We consider this feature important. Without it using Matrix for collaboration in a business setting is cumbersome.
E.g when:

  • Bringing new team members to a project which is discussed in a room
  • Have a co-worker cover your tasks when on vacation
  • ++

What needs to happen to get this MSC moving again, so it can be merged, and find its way to clients and servers?

@jeudesprits
Copy link

I agree with what has been said above. We also chose Matrix but didn’t pay much attention to this issue at the beginning, and now we really need this feature. It would be great to hear some timelines, if possible.

@thoraj
Copy link

thoraj commented Sep 5, 2024

A guesstimate for when the security disclosure will be made would be nice. Depending on the potential attack vector the risks may be mitigated or accepted.

Without the disclosure we don't know.

@mpeter50
Copy link

mpeter50 commented Sep 5, 2024

I agree that this feature is very important, but @thoraj , did you intend your last comment to be sent here? If so, what kind of security disclosure do you mean? As I understand, a vulnerability was not a topic here.

@thoraj
Copy link

thoraj commented Sep 5, 2024

@mpeter50 Sorry for not providing context.

My reference to a disclosure is based on a comment made in #e2e:matrix.org today. Apparently the MSC is blocked awaiting a security disclosure and having the issue resolved.

@dkasak
Copy link
Member Author

dkasak commented Sep 5, 2024

We're working on this now, so the disclosure should come in the next couple of weeks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
App: Android App: iOS O-Frequent Affects or can be seen by most users regularly or impacts most users' first experience T-Enhancement T-Epic Issue is at Epic level Team: Crypto Z-Chronic
Projects
None yet
Development

No branches or pull requests

10 participants