element-hq / element-hq/synapse

check_event_for_spam doesn't get called for new local member events

Open
#10,641 0 comments 0 reactions 0 assignees View on GitHub
T-Enhancement
Dominant language
Python
Stars
4.6k
Forks
600
Avg merge
5d 22h
Merged PRs (30d)
51

Description

This issue has been migrated from [#10641](https://github.com/matrix-org/synapse/issues/10641).

---

Currently `check_event_for_spam` only gets called for local events in `create_and_send_nonmember_event`, meaning it doesn't get called for new `m.room.member` events, which looks like a thinko. Maybe it should instead be called in `create_new_client_event` which is where we also call `check_event_allowed`, or in `handle_new_client_event`.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.