Fix fetching public rooms over federation (#12410)

Broke by #12364
matrix-org · Apr 7, 2022 · 36af768 · 36af768
1 parent 1a90c1e
commit 36af768
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/changelog.d/12410.bugfix b/changelog.d/12410.bugfix
@@ -0,0 +1 @@
+Fix a spec compliance issue where requests to the `/publicRooms` federation API would specify `limit` as a string.
diff --git a/synapse/federation/transport/client.py b/synapse/federation/transport/client.py
@@ -509,7 +509,7 @@ async def get_public_rooms(
             if third_party_instance_id:
                 args["third_party_instance_id"] = (third_party_instance_id,)
             if limit:
-                args["limit"] = [limit]
+                args["limit"] = [str(limit)]
             if since_token:
                 args["since"] = [since_token]