Skip to content

MSC4294: Ignore and mass ignore invites#4294

Closed
grinapo wants to merge 5 commits intomatrix-org:mainfrom
grinapo:master
Closed

MSC4294: Ignore and mass ignore invites#4294
grinapo wants to merge 5 commits intomatrix-org:mainfrom
grinapo:master

Conversation

@grinapo
Copy link
Copy Markdown
Contributor

@grinapo grinapo commented May 29, 2025

Rendered


SCT Stuff:

No MSC checklist

FCP close tickyboxes

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

@richvdh
Copy link
Copy Markdown
Member

richvdh commented Nov 18, 2025

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.

Agreed, and I don't see any counterarguments forthcoming.

In the interest of reducing the number of active MSCs in this field:

@mscbot fcp close

@mscbot
Copy link
Copy Markdown
Collaborator

mscbot commented Nov 18, 2025

Team member @richvdh has proposed to close this. The next step is review by the rest of the tagged people:

Once at least 75% of reviewers approve (and there are no outstanding concerns), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for information about what commands tagged team members can give me.

@mscbot mscbot added proposed-final-comment-period Currently awaiting signoff of a majority of team members in order to enter the final comment period. disposition-close labels Nov 18, 2025
@github-project-automation github-project-automation Bot moved this to Tracking for review in Spec Core Team Workflow Nov 18, 2025
@turt2live turt2live moved this from Tracking for review to Ready for FCP ticks in Spec Core Team Workflow Nov 18, 2025
@turt2live turt2live added the 00-weekly-pings Tracking for weekly pings in the SCT office. 00 to make it first in the labels list. label Nov 18, 2025
@mscbot
Copy link
Copy Markdown
Collaborator

mscbot commented Dec 2, 2025

🔔 This is now entering its final comment period, as per the review above. 🔔

@mscbot mscbot added final-comment-period This MSC has entered a final comment period in interest to approval, postpone, or delete in 5 days. and removed proposed-final-comment-period Currently awaiting signoff of a majority of team members in order to enter the final comment period. labels Dec 2, 2025
@turt2live turt2live moved this from Ready for FCP ticks to In FCP in Spec Core Team Workflow Dec 2, 2025
@turt2live turt2live removed the 00-weekly-pings Tracking for weekly pings in the SCT office. 00 to make it first in the labels list. label Dec 2, 2025
@mscbot
Copy link
Copy Markdown
Collaborator

mscbot commented Dec 7, 2025

The final comment period, with a disposition to close, as per the review above, is now complete.

@mscbot mscbot closed this Dec 7, 2025
@mscbot mscbot added finished-final-comment-period and removed disposition-close final-comment-period This MSC has entered a final comment period in interest to approval, postpone, or delete in 5 days. labels Dec 7, 2025
@turt2live turt2live moved this from In FCP to Merged in Spec Core Team Workflow Dec 8, 2025
@turt2live turt2live added the rejected A proposal which has been rejected for inclusion in the spec label Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

client-server Client-Server API finished-final-comment-period 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 rejected A proposal which has been rejected for inclusion in the spec safety

Projects

Status: Merged/Done

Development

Successfully merging this pull request may close these issues.

4 participants