diff options
| author | David Robertson <davidr@element.io> | 2023-03-08 18:14:17 +0000 |
|---|---|---|
| committer | David Robertson <davidr@element.io> | 2023-03-08 19:21:29 +0000 |
| commit | 0aa020145267829d1ccf9dff6200ebaef4c07c7a (patch) | |
| tree | dfa2516bc57745b0c868d9e5af38787e2842a759 | |
| parent | Flip logic and provide better name (diff) | |
| download | synapse-0aa020145267829d1ccf9dff6200ebaef4c07c7a.tar.xz | |
Unconditionally omit remote events in partial state rooms
I can't see why erased senders makes a difference here?
| -rw-r--r-- | synapse/visibility.py | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/synapse/visibility.py b/synapse/visibility.py index cb9a2d1e0a..92955fc2ce 100644 --- a/synapse/visibility.py +++ b/synapse/visibility.py @@ -643,14 +643,13 @@ async def filter_events_for_server( # this check but would base the filtering on an outdated view of the membership events. partial_state_invisible_events = set() - if filter_out_erased_senders: - for e in events: - sender_domain = get_domain_from_id(e.sender) - if ( - sender_domain != local_server_name - and await storage.main.is_partial_state_room(e.room_id) - ): - partial_state_invisible_events.add(e) + for e in events: + sender_domain = get_domain_from_id(e.sender) + if ( + sender_domain != local_server_name + and await storage.main.is_partial_state_room(e.room_id) + ): + partial_state_invisible_events.add(e) # Let's check to see if all the events have a history visibility # of "shared" or "world_readable". If that's the case then we don't |