summary refs log tree commit diff
diff options
context:
space:
mode:
authorreivilibre <oliverw@matrix.org>2022-04-05 12:45:36 +0100
committerGitHub <noreply@github.com>2022-04-05 12:45:36 +0100
commit42d8710f38d465904dfe2823d132c299b94f7a3a (patch)
tree4afc3df7a16c6bced7da790866589820b1a08f0d
parentAllow specifying the Postgres database's port when running unit tests with Po... (diff)
downloadsynapse-42d8710f38d465904dfe2823d132c299b94f7a3a.tar.xz
Fix a spec compliance issue where requests to the `/publicRooms` federation API would specify `limit` as a string. (#12364)
-rw-r--r--changelog.d/12364.bugfix1
-rw-r--r--synapse/federation/transport/client.py4
2 files changed, 3 insertions, 2 deletions
diff --git a/changelog.d/12364.bugfix b/changelog.d/12364.bugfix
new file mode 100644
index 0000000000..d042425d20
--- /dev/null
+++ b/changelog.d/12364.bugfix
@@ -0,0 +1 @@
+Fix a spec compliance issue where requests to the `/publicRooms` federation API would specify `limit` as a string.
\ No newline at end of file
diff --git a/synapse/federation/transport/client.py b/synapse/federation/transport/client.py
index de6e5f44fe..d4b3cb3e98 100644
--- a/synapse/federation/transport/client.py
+++ b/synapse/federation/transport/client.py
@@ -481,7 +481,7 @@ class TransportLayerClient:
             if third_party_instance_id:
                 data["third_party_instance_id"] = third_party_instance_id
             if limit:
-                data["limit"] = str(limit)
+                data["limit"] = limit
             if since_token:
                 data["since"] = since_token
 
@@ -509,7 +509,7 @@ class TransportLayerClient:
             if third_party_instance_id:
                 args["third_party_instance_id"] = (third_party_instance_id,)
             if limit:
-                args["limit"] = [str(limit)]
+                args["limit"] = [limit]
             if since_token:
                 args["since"] = [since_token]