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/rest/admin/statistics.py | |
| 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/rest/admin/statistics.py')
| -rw-r--r-- | synapse/rest/admin/statistics.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/synapse/rest/admin/statistics.py b/synapse/rest/admin/statistics.py index dc27a41dd9..0adc5b7005 100644 --- a/synapse/rest/admin/statistics.py +++ b/synapse/rest/admin/statistics.py @@ -63,10 +63,10 @@ class UserMediaStatisticsRestServlet(RestServlet): ), ) - start = parse_integer(request, "from", default=0, negative=False) - limit = parse_integer(request, "limit", default=100, negative=False) - from_ts = parse_integer(request, "from_ts", default=0, negative=False) - until_ts = parse_integer(request, "until_ts", negative=False) + start = parse_integer(request, "from", default=0) + limit = parse_integer(request, "limit", default=100) + from_ts = parse_integer(request, "from_ts", default=0) + until_ts = parse_integer(request, "until_ts") if until_ts is not None: if until_ts <= from_ts: |