diff options
| author | Patrick Cloke <clokep@users.noreply.github.com> | 2022-10-14 08:30:05 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-14 12:30:05 +0000 |
| commit | 126a15794c95002560709283640ad412636b29b8 (patch) | |
| tree | dca3e2380f4bec49b5dcff4407bc6405eef05692 /synapse/handlers/presence.py | |
| parent | Set LD_PRELOAD to load jemalloc in Dockerfile-workers. (#14182) (diff) | |
| download | synapse-126a15794c95002560709283640ad412636b29b8.tar.xz | |
Do not allow a None-limit on PaginationConfig. (#14146)
The callers either set a default limit or manually handle a None-limit later on (by setting a default value). Update the callers to always instantiate PaginationConfig with a default limit and then assume the limit is non-None.
Diffstat (limited to 'synapse/handlers/presence.py')
| -rw-r--r-- | synapse/handlers/presence.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/synapse/handlers/presence.py b/synapse/handlers/presence.py index 4e575ffbaa..2670e561d7 100644 --- a/synapse/handlers/presence.py +++ b/synapse/handlers/presence.py @@ -1596,7 +1596,9 @@ class PresenceEventSource(EventSource[int, UserPresenceState]): self, user: UserID, from_key: Optional[int], - limit: Optional[int] = None, + # Having a default limit doesn't match the EventSource API, but some + # callers do not provide it. It is unused in this class. + limit: int = 0, room_ids: Optional[Collection[str]] = None, is_guest: bool = False, explicit_room_id: Optional[str] = None, |