Back to set.remove() to protect us from our own mistakes

See https://github.com/element-hq/synapse/pull/17335#discussion_r1646701974
This commit is contained in:
Eric Eastwood 2024-06-19 17:40:12 -05:00
parent 02fcf39c00
commit 62157ce2df

View file

@ -591,7 +591,7 @@ class SlidingSyncHandler:
if (filters.is_encrypted and not is_encrypted) or (
not filters.is_encrypted and is_encrypted
):
filtered_room_id_set.discard(room_id)
filtered_room_id_set.remove(room_id)
# Filter for rooms that the user has been invited to
if filters.is_invite is not None:
@ -607,7 +607,7 @@ class SlidingSyncHandler:
not filters.is_invite
and room_for_user.membership == Membership.INVITE
):
filtered_room_id_set.discard(room_id)
filtered_room_id_set.remove(room_id)
if filters.room_types:
raise NotImplementedError()