diff options
author | Erik Johnston <erik@matrix.org> | 2018-10-01 14:19:36 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2018-10-01 14:19:40 +0100 |
commit | 82f922b4af8d41e15484e1913775d234c548d9f2 (patch) | |
tree | e4965749d387feccca8facc55b7dbf309af700af | |
parent | Merge pull request #3933 from matrix-org/erikj/destination_retry_cache (diff) | |
download | synapse-82f922b4af8d41e15484e1913775d234c548d9f2.tar.xz |
Fix lazy loaded sync with rejected state events
In particular, we assume that the name and canonical alias events in the state have not been rejected. In practice this may not be the case (though we should probably think about fixing that) so lets ensure that we gracefully handle that case, rather than 404'ing the sync request like we do now.
-rw-r--r-- | synapse/handlers/sync.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/synapse/handlers/sync.py b/synapse/handlers/sync.py index c7d69d9d80..67b8ca28c7 100644 --- a/synapse/handlers/sync.py +++ b/synapse/handlers/sync.py @@ -567,13 +567,13 @@ class SyncHandler(object): # be a valid name or canonical_alias - i.e. we're checking that they # haven't been "deleted" by blatting {} over the top. if name_id: - name = yield self.store.get_event(name_id, allow_none=False) + name = yield self.store.get_event(name_id, allow_none=True) if name and name.content: defer.returnValue(summary) if canonical_alias_id: canonical_alias = yield self.store.get_event( - canonical_alias_id, allow_none=False, + canonical_alias_id, allow_none=True, ) if canonical_alias and canonical_alias.content: defer.returnValue(summary) |