diff options
author | David Baker <dave@matrix.org> | 2016-04-07 17:09:36 +0100 |
---|---|---|
committer | David Baker <dave@matrix.org> | 2016-04-07 17:09:36 +0100 |
commit | e1e042f2a1bc489c922f0deccfac54572788c933 (patch) | |
tree | d85dcd79334b9ba7549c80363e92b3a708218fba | |
parent | pep8 (diff) | |
download | synapse-e1e042f2a1bc489c922f0deccfac54572788c933.tar.xz |
Add comments on min_stream_id
saying that the min stream id won't be completely accurate all the time
Diffstat (limited to '')
-rw-r--r-- | synapse/handlers/receipts.py | 1 | ||||
-rw-r--r-- | synapse/push/httppusher.py | 2 |
2 files changed, 3 insertions, 0 deletions
diff --git a/synapse/handlers/receipts.py b/synapse/handlers/receipts.py index 26b0368080..a390a1b8bd 100644 --- a/synapse/handlers/receipts.py +++ b/synapse/handlers/receipts.py @@ -111,6 +111,7 @@ class ReceiptsHandler(BaseHandler): self.notifier.on_new_event( "receipt_key", max_batch_id, rooms=affected_room_ids ) + # Note that the min here shouldn't be relied upon to be accurate. self.hs.get_pusherpool().on_new_receipts( min_batch_id, max_batch_id, affected_room_ids ) diff --git a/synapse/push/httppusher.py b/synapse/push/httppusher.py index cc030a57a0..0a1d3817de 100644 --- a/synapse/push/httppusher.py +++ b/synapse/push/httppusher.py @@ -87,6 +87,8 @@ class HttpPusher(object): @defer.inlineCallbacks def on_new_receipts(self, min_stream_id, max_stream_id): + # Note that the min here shouldn't be relied upon to be accurate. + # We could check the receipts are actually m.read receipts here, # but currently that's the only type of receipt anyway... badge = yield push_tools.get_badge_count(self.hs, self.user_id) |