summary refs log tree commit diff
path: root/synapse/storage/state.py
diff options
context:
space:
mode:
authorRichard van der Hoff <1389908+richvdh@users.noreply.github.com>2022-08-11 11:42:24 +0100
committerGitHub <noreply@github.com>2022-08-11 10:42:24 +0000
commit507c1cb3309e989d84ec3ff9557a96ae1fc7f369 (patch)
treed0ff6f14715ae9df656fbaff9df2d5974770b67b /synapse/storage/state.py
parentUse literals in place of `HTTPStatus` constants in tests (#13488) (diff)
downloadsynapse-507c1cb3309e989d84ec3ff9557a96ae1fc7f369.tar.xz
Update the rejected state of events during resync (#13459)
Events can be un-rejected or newly-rejected during resync, so ensure we update
the database and caches when that happens.
Diffstat (limited to '')
-rw-r--r--synapse/storage/state.py9
1 files changed, 0 insertions, 9 deletions
diff --git a/synapse/storage/state.py b/synapse/storage/state.py
index af3bab2c15..0004d955b4 100644
--- a/synapse/storage/state.py
+++ b/synapse/storage/state.py
@@ -539,15 +539,6 @@ class StateFilter:
             is_mine_id: a callable which confirms if a given state_key matches a mxid
                of a local user
         """
-
-        # TODO(faster_joins): it's not entirely clear that this is safe. In particular,
-        #  there may be circumstances in which we return a piece of state that, once we
-        #  resync the state, we discover is invalid. For example: if it turns out that
-        #  the sender of a piece of state wasn't actually in the room, then clearly that
-        #  state shouldn't have been returned.
-        #  We should at least add some tests around this to see what happens.
-        #  https://github.com/matrix-org/synapse/issues/13006
-
         # if we haven't requested membership events, then it depends on the value of
         # 'include_others'
         if EventTypes.Member not in self.types: