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

MSC3786: Add a default push rule to ignore m.room.server_acl events #3786

Merged
merged 16 commits into from
Jul 17, 2022
Merged

MSC3786: Add a default push rule to ignore m.room.server_acl events #3786

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
c109875
Add a default push rule to ignore `m.room.server_acl` events
SimonBrandner Apr 30, 2022
0c78f6e
Update MSC number
SimonBrandner Apr 30, 2022
94ac57a
Add prefix `.`
SimonBrandner Apr 30, 2022
53d42f7
Link to spec
SimonBrandner May 4, 2022
c86713e
Be more explicit about why the rule is needed
SimonBrandner May 4, 2022
b28af33
Clarify push rule ordering
SimonBrandner May 4, 2022
5d47fe4
Don't rely on unmerged MSCs for ordering
SimonBrandner May 5, 2022
0170475
Be even more explicit about why we need this
SimonBrandner May 5, 2022
7307bcb
Improve wording
SimonBrandner May 23, 2022
3cbd4d2
Improve wording
SimonBrandner May 23, 2022
d9bcf88
Improve wording
SimonBrandner May 23, 2022
a0baa9d
Add missing period
SimonBrandner May 23, 2022
bb6664a
Add spec link
SimonBrandner May 23, 2022
af56f02
Remove `when merged`
SimonBrandner May 23, 2022
bc4440d
Remove `dont_notify`
SimonBrandner May 23, 2022
233efdb
Check `state_key`
SimonBrandner Jun 1, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions proposals/3786-acl-notifs.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,11 @@

`m.room.server_acl` events allow for expressing which servers can participate in
a room. Often during spam attacks, these events are sent quite frequently, which
causes the users to be overwhelmed by notifications. (See
causes the users to be overwhelmed by notifications, if they have the room rule
set to `notify` (`All messages` in Element). (See
<https://github.com/vector-im/element-web/issues/20788>) As this is very
unideal, this MSC proposes a new push rule to avoid this.
unideal, this MSC proposes a new push rule to avoid this. It is analogues
to [MSC2153](https://github.com/matrix-org/matrix-spec-proposals/pull/2153).
richvdh marked this conversation as resolved.
Show resolved Hide resolved

## Proposal

Expand Down