Merge pull request #5324 from matrix-org/erikj/ignore_null

Ignore room state with null bytes in for room stats

changelog.d/5324.feature

@@ -0,0 +1 @@
+Synapse now more efficiently collates room statistics.

synapse/storage/stats.py

@@ -328,6 +328,22 @@ def update_room_state(self, room_id, fields):
             room_id (str)
             fields (dict[str:Any])
         """
+
+        # For whatever reason some of the fields may contain null bytes, which
+        # postgres isn't a fan of, so we replace those fields with null.
+        for col in (
+            "join_rules",
+            "history_visibility",
+            "encryption",
+            "name",
+            "topic",
+            "avatar",
+            "canonical_alias"
+        ):
+            field = fields.get(col)
+            if field and "\0" in field:
+                fields[col] = None
+
         return self._simple_upsert(
             table="room_state",
             keyvalues={"room_id": room_id},