Skip to content

MSC4294: Ignore and mass ignore invites #4294

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

grinapo
Copy link
Contributor

@grinapo grinapo commented May 29, 2025

@grinapo grinapo changed the title Add ignore invites proposal. MSC4294: Add ignore invites proposal. May 29, 2025
grinapo added 2 commits May 29, 2025 11:11
Now that I know the number.
Better phrasing to distinguish between generic ignore and invite ignore.
Server behaviour specified.
@grinapo
Copy link
Contributor Author

grinapo commented May 29, 2025

There is a long history of ignored open ignore invites proposals. There is also a summary MSC to list all the MSCs about the topic.

All of these proposals are open and have been for more than 5 years. They are fairly complex, and handle various complex combinations of ignores and allowlists, for users, servers, and patterns, as well as some preauthorisation.

This current proposal is deliberately very simple, using the already established ignore machanism, which should be easy to implement using parts of the same code. It is hoped that it will not take years to be accepted, since the problem is acute.

cc: @Johennes

@turt2live turt2live changed the title MSC4294: Add ignore invites proposal. MSC4294: Ignore and mass ignore invites May 29, 2025
@turt2live turt2live added proposal A matrix spec change proposal client-server Client-Server API kind:feature MSC for not-core and not-maintenance stuff needs-implementation This MSC does not have a qualifying implementation for the SCT to review. The MSC cannot enter FCP. safety labels May 29, 2025
Copy link
Member

Choose a reason for hiding this comment

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

Implementation requirements:

  • Client
  • Server

@turt2live
Copy link
Member

Your MSC currently uses the master branch of your fork, which has already shown to create compatibility issues with another MSC of yours: #2063 (comment)

Please consider switching the branch to a feature branch instead of master.

@grinapo
Copy link
Contributor Author

grinapo commented May 29, 2025

@turt2live
Please consider switching the branch to a feature branch instead of master.

(I hate git)
What magical commands shall be casted upon the repo which would move it onto a feature branch while keeping this PR intact?

@turt2live
Copy link
Member

The internet is telling me it's not possible to change the source branch after the PR is opened, but I'm pretty sure we've done that before. For all the hassle, a new PR is probably easiest and causes no harm.

Copy link
Member

Choose a reason for hiding this comment

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

Overall, I feel this MSC is in direct competition with #4155 which has a number of implementations and experimentation. If there's parts that 4155 doesn't cover, this MSC would be best positioned to propose extensions to 4155 so that it may protect more users.

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:feature MSC for not-core and not-maintenance stuff 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 safety
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants