Merge pull request #2014 from Half-Shot/hs/fix-appservice-presence

Add fallback to last_active_ts if it beats the last sync time on a presence timeout.
matrix-org · Mar 15, 2017 · 54d2b7e · 54d2b7e
2 parents 9d52719 + c144292
commit 54d2b7e
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/synapse/handlers/presence.py b/synapse/handlers/presence.py
@@ -1174,7 +1174,10 @@ def handle_timeout(state, is_mine, syncing_user_ids, now):
         # If there are have been no sync for a while (and none ongoing),
         # set presence to offline
         if user_id not in syncing_user_ids:
-            if now - state.last_user_sync_ts > SYNC_ONLINE_TIMEOUT:
+            # If the user has done something recently but hasn't synced,
+            # don't set them as offline.
+            sync_or_active = max(state.last_user_sync_ts, state.last_active_ts)
+            if now - sync_or_active > SYNC_ONLINE_TIMEOUT:
                 state = state.copy_and_replace(
                     state=PresenceState.OFFLINE,
                     status_msg=None,

diff --git a/tests/handlers/test_presence.py b/tests/handlers/test_presence.py
@@ -324,7 +324,7 @@ def test_sync_timeout(self):
         state = UserPresenceState.default(user_id)
         state = state.copy_and_replace(
             state=PresenceState.ONLINE,
-            last_active_ts=now,
+            last_active_ts=0,
             last_user_sync_ts=now - SYNC_ONLINE_TIMEOUT - 1,
         )