diff options
author | Erik Johnston <erik@matrix.org> | 2016-01-14 10:22:02 +0000 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2016-01-14 10:22:02 +0000 |
commit | 339c8f013319f812629bb9a294fb235ada1e888d (patch) | |
tree | cb3dbea8f960a85b34b7f4e9901dcf282df27f8d /synapse/streams/config.py | |
parent | Use logger not logging (diff) | |
download | synapse-339c8f013319f812629bb9a294fb235ada1e888d.tar.xz |
Clamp pagination limits to at most 1000
Diffstat (limited to 'synapse/streams/config.py')
-rw-r--r-- | synapse/streams/config.py | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/synapse/streams/config.py b/synapse/streams/config.py index 8c082bf4e0..4f089bfb94 100644 --- a/synapse/streams/config.py +++ b/synapse/streams/config.py @@ -22,6 +22,9 @@ import logging logger = logging.getLogger(__name__) +MAX_LIMIT = 1000 + + class SourcePaginationConfig(object): """A configuration object which stores pagination parameters for a @@ -32,7 +35,7 @@ class SourcePaginationConfig(object): self.from_key = from_key self.to_key = to_key self.direction = 'f' if direction == 'f' else 'b' - self.limit = int(limit) if limit is not None else None + self.limit = min(int(limit), MAX_LIMIT) if limit is not None else None def __repr__(self): return ( @@ -49,7 +52,7 @@ class PaginationConfig(object): self.from_token = from_token self.to_token = to_token self.direction = 'f' if direction == 'f' else 'b' - self.limit = int(limit) if limit is not None else None + self.limit = min(int(limit), MAX_LIMIT) if limit is not None else None @classmethod def from_request(cls, request, raise_invalid_params=True, |