diff options
| author | Neil Alexander <neilalexander@users.noreply.github.com> | 2021-08-27 16:50:30 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-08-27 16:50:30 +0100 |
| commit | 51b119107c972007f5a1bc626758a6229ece14db (patch) | |
| tree | 5a19d194f2ab3c664195bf50d0c7286a705aae85 /roomserver | |
| parent | 2dd5fd1fd676c570a13c2ed062fa47a026ca826f (diff) | |
Don't return nonsense canonical room aliases in the public rooms responses (#1992)
Diffstat (limited to 'roomserver')
| -rw-r--r-- | roomserver/api/wrapper.go | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/roomserver/api/wrapper.go b/roomserver/api/wrapper.go index 2ebe2f64..de66df80 100644 --- a/roomserver/api/wrapper.go +++ b/roomserver/api/wrapper.go @@ -215,7 +215,9 @@ func PopulatePublicRooms(ctx context.Context, roomIDs []string, rsAPI Roomserver case topicTuple: pub.Topic = contentVal case canonicalTuple: - pub.CanonicalAlias = contentVal + if _, _, err := gomatrixserverlib.SplitID('#', contentVal); err == nil { + pub.CanonicalAlias = contentVal + } case visibilityTuple: pub.WorldReadable = contentVal == "world_readable" // need both of these to determine whether guests can join |