matrix-org / matrix-org/mjolnir
Imported server bans get synced individually
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 445
- Forks
- 75
- Avg merge
- 2d 44m
- Merged PRs (30d)
- 3
Description
When importing a large ban list (e.g. #matrix-org-coc-bl:matrix.org), each server ban causes the ACLs to be updated in every protected room. This means for 100+ server bans, the ACLs are updated 100+ times. It would be great for the ACL updates to happen all at once. So all bans are imported and then the ACLs are updated once. This would reduce event spam in protected rooms and reduce the overall number of events generated.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the server-ban import path and the ACL update operation it triggers. Reproduce the import with a large ban list such as #matrix-org-coc-bl:matrix.org, then verify that all bans are imported before protected-room ACLs are updated once rather than once per ban.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- authorization
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100