Skip to content
Permalink
Browse files

Merge pull request #5089 from dnaf/m-heroes-empty-room-name

Make /sync return heroes if room name or canonical alias are empty
  • Loading branch information...
babolivier committed Jun 6, 2019
2 parents cb3b381 + 7898a1a commit 8f06344e11027dceb9634a30504aa81e338b580f
Showing with 5 additions and 5 deletions.
  1. +1 −0 changelog.d/5089.bugfix
  2. +4 −5 synapse/handlers/sync.py
@@ -0,0 +1 @@
Fixes client-server API not sending "m.heroes" to lazy-load /sync requests when a rooms name or its canonical alias are empty. Thanks to @dnaf for this work!
@@ -583,19 +583,18 @@ def compute_summary(self, room_id, sync_config, batch, state, now_token):
)

# if the room has a name or canonical_alias set, we can skip
# calculating heroes. we assume that if the event has contents, it'll
# be a valid name or canonical_alias - i.e. we're checking that they
# haven't been "deleted" by blatting {} over the top.
# calculating heroes. Empty strings are falsey, so we check
# for the "name" value and default to an empty string.
if name_id:
name = yield self.store.get_event(name_id, allow_none=True)
if name and name.content:
if name and name.content.get("name"):
defer.returnValue(summary)

if canonical_alias_id:
canonical_alias = yield self.store.get_event(
canonical_alias_id, allow_none=True,
)
if canonical_alias and canonical_alias.content:
if canonical_alias and canonical_alias.content.get("alias"):
defer.returnValue(summary)

me = sync_config.user.to_string()

0 comments on commit 8f06344

Please sign in to comment.
You can’t perform that action at this time.