summary refs log tree commit diff
path: root/synapse/handlers/receipts.py
diff options
context:
space:
mode:
authorRichard van der Hoff <github@rvanderhoff.org.uk>2016-12-08 12:34:57 +0000
committerGitHub <noreply@github.com>2016-12-08 12:34:57 +0000
commitda13b4aa861ab1596c7a08be8fecaf15f4f9db7a (patch)
tree252fbf11b25a154cbb5e7cf5e544746b03986dc6 /synapse/handlers/receipts.py
parentMerge pull request #1672 from williamleuschner/develop (diff)
parentFix ignored read-receipts (diff)
downloadsynapse-da13b4aa861ab1596c7a08be8fecaf15f4f9db7a.tar.xz
Merge pull request #1678 from matrix-org/rav/fix_receipt_notifications
Read-receipt fixes
Diffstat (limited to 'synapse/handlers/receipts.py')
-rw-r--r--synapse/handlers/receipts.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/synapse/handlers/receipts.py b/synapse/handlers/receipts.py
index 916e80a48e..50aa513935 100644
--- a/synapse/handlers/receipts.py
+++ b/synapse/handlers/receipts.py
@@ -100,7 +100,7 @@ class ReceiptsHandler(BaseHandler):
 
             if not res:
                 # res will be None if this read receipt is 'old'
-                defer.returnValue(False)
+                continue
 
             stream_id, max_persisted_id = res
 
@@ -109,6 +109,10 @@ class ReceiptsHandler(BaseHandler):
             if max_batch_id is None or max_persisted_id > max_batch_id:
                 max_batch_id = max_persisted_id
 
+        if min_batch_id is None:
+            # no new receipts
+            defer.returnValue(False)
+
         affected_room_ids = list(set([r["room_id"] for r in receipts]))
 
         with PreserveLoggingContext():