diff options
| author | Denis Kasak <dkasak@termina.org.uk> | 2024-06-24 15:12:14 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-06-24 15:12:14 +0200 |
| commit | 700d2cc4a0d457642edb43bc3714d212f15d797f (patch) | |
| tree | fc338a7d0cd2649b3dba0992e9e8c59e2ac1c349 /synapse/streams | |
| parent | Bump lazy_static from 1.4.0 to 1.5.0 (#17355) (diff) | |
| download | synapse-700d2cc4a0d457642edb43bc3714d212f15d797f.tar.xz | |
Tidy up integer parsing (#17339)
The parse_integer function was previously made to reject negative values by default in https://github.com/element-hq/synapse/pull/16920, but the documentation stated otherwise. This fixes the documentation and also: - Removes explicit negative=False parameters from call sites. - Brings the negative default of parse_integer_from_args in alignment with parse_integer.
Diffstat (limited to 'synapse/streams')
| -rw-r--r-- | synapse/streams/config.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/synapse/streams/config.py b/synapse/streams/config.py index eeafe889de..9fee5bfb92 100644 --- a/synapse/streams/config.py +++ b/synapse/streams/config.py @@ -75,9 +75,6 @@ class PaginationConfig: raise SynapseError(400, "'to' parameter is invalid") limit = parse_integer(request, "limit", default=default_limit) - if limit < 0: - raise SynapseError(400, "Limit must be 0 or above") - limit = min(limit, MAX_LIMIT) try: |