diff options
author | Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> | 2019-10-08 13:51:25 +0100 |
---|---|---|
committer | Amber Brown <hawkowl@atleastfornow.net> | 2019-10-08 13:51:25 +0100 |
commit | ea7d938bca2d5fa0d6a54412ecdf036c5a3fc3a7 (patch) | |
tree | 5a6d38cadeebe9d3d904fff51f37c5f0a4c13c65 | |
parent | Fix /federation/v1/state for recent room versions (#6170) (diff) | |
download | synapse-ea7d938bca2d5fa0d6a54412ecdf036c5a3fc3a7.tar.xz |
Remove unused public room list timeout param (#6179)
* Remove unused public room list timeout param * Add changelog
-rw-r--r-- | changelog.d/6179.misc | 1 | ||||
-rw-r--r-- | synapse/handlers/room_list.py | 13 |
2 files changed, 2 insertions, 12 deletions
diff --git a/changelog.d/6179.misc b/changelog.d/6179.misc new file mode 100644 index 0000000000..01c4e71ea3 --- /dev/null +++ b/changelog.d/6179.misc @@ -0,0 +1 @@ +Remove unused `timeout` parameter from `_get_public_room_list`. \ No newline at end of file diff --git a/synapse/handlers/room_list.py b/synapse/handlers/room_list.py index cfed344d4d..c615206df1 100644 --- a/synapse/handlers/room_list.py +++ b/synapse/handlers/room_list.py @@ -88,16 +88,8 @@ class RoomListHandler(BaseHandler): # appservice specific lists. logger.info("Bypassing cache as search request.") - # XXX: Quick hack to stop room directory queries taking too long. - # Timeout request after 60s. Probably want a more fundamental - # solution at some point - timeout = self.clock.time() + 60 return self._get_public_room_list( - limit, - since_token, - search_filter, - network_tuple=network_tuple, - timeout=timeout, + limit, since_token, search_filter, network_tuple=network_tuple ) key = (limit, since_token, network_tuple) @@ -118,7 +110,6 @@ class RoomListHandler(BaseHandler): search_filter=None, network_tuple=EMPTY_THIRD_PARTY_ID, from_federation=False, - timeout=None, ): """Generate a public room list. Args: @@ -131,8 +122,6 @@ class RoomListHandler(BaseHandler): Setting to None returns all public rooms across all lists. from_federation (bool): Whether this request originated from a federating server or a client. Used for room filtering. - timeout (int|None): Amount of seconds to wait for a response before - timing out. TODO """ # Pagination tokens work by storing the room ID sent in the last batch, |