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

MSC3325: Upgrading invite-only rooms #3325

Open
wants to merge 4 commits into
base: old_master
Choose a base branch
from

Conversation

uhoreg
Copy link
Member

@uhoreg uhoreg commented Aug 10, 2021

@uhoreg uhoreg changed the title MSCxxxx: Upgrading invite-only rooms MSC3325: Upgrading invite-only rooms Aug 10, 2021
@uhoreg uhoreg added client-server Client-Server API kind:maintenance MSC which clarifies/updates existing spec needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. proposal A matrix spec change proposal labels Aug 10, 2021
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a much cleaner solution


## Potential issues

None?
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you upgrade a knocking room like this, you lose the ability to knock, thanks to us having made knocking & restricted access mutually exclusive.

Also, it's not clear whether one should remove the restricted access rule down the line or not, or whether you end up stuck in restricted mode forever.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Valid point, but the MSC seems to concern itself with invite-only rooms rather than non-public rooms

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added a note about the first point.

For the second point, I think we just leave it to the room admins to change the join rule whenever they feel it's appropriate to do so.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there should be a client impl recommendation that clients label the pre-upgraded room specifically in the restricted join rule settings rather than generically by name+avatar as they may do otherwise. E.g members of a prior version of this room are able to join without an invitation or similar

@@ -0,0 +1,81 @@
# MSC3325: Upgrading invite-only rooms

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the advantage of this approach? There are disadvantages listed in this proposal but no advantages.

version allows both to be used, then this proposal can be extended at that
time.

If the room is encrypted, users will not receive the keys to decrypt messages
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a pretty huge issue imho. I'm just wondering how to soon upgrade some encrypted community rooms I'm in that have 800+ members. Most of the members only check messages rarely, which means that a lot of the members would lose messages for a large part of time after the upgrade is triggered up until they join. I'm not entirely sure if there is anything that can be improved regarding this in this MSC, it's probably best to go via the invite way for encrypted invite only rooms with room history set to "invited", but voicing as a +1 to potential issue.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think MSC3061 could be of some help

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client-server Client-Server API kind:maintenance MSC which clarifies/updates existing spec needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. proposal A matrix spec change proposal
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants