diff options
author | Erik Johnston <erik@matrix.org> | 2023-03-29 12:07:14 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-03-29 12:07:14 +0100 |
commit | 78cdb72cd6b0e007c314d9fed9f629dfc5b937a6 (patch) | |
tree | d62e56a47d56a3f575eb3aa87e49202ae133873a /tests/handlers | |
parent | doc: fix account login requests ratelimit defaults typo (#15341) (diff) | |
download | synapse-78cdb72cd6b0e007c314d9fed9f629dfc5b937a6.tar.xz |
Delete stale non-e2e devices for users, take 3 (#15183)
This should help reduce the number of devices e.g. simple bots the repeatedly login rack up. We only delete non-e2e devices as they should be safe to delete, whereas if we delete e2e devices for a user we may accidentally break their ability to receive e2e keys for a message.
Diffstat (limited to 'tests/handlers')
-rw-r--r-- | tests/handlers/test_admin.py | 2 | ||||
-rw-r--r-- | tests/handlers/test_device.py | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/tests/handlers/test_admin.py b/tests/handlers/test_admin.py index 5569ccef8a..f0ba3775c8 100644 --- a/tests/handlers/test_admin.py +++ b/tests/handlers/test_admin.py @@ -272,7 +272,7 @@ class ExfiltrateData(unittest.HomeserverTestCase): self.assertIn("device_id", args[0][0]) self.assertIsNone(args[0][0]["display_name"]) self.assertIsNone(args[0][0]["last_seen_user_agent"]) - self.assertIsNone(args[0][0]["last_seen_ts"]) + self.assertEqual(args[0][0]["last_seen_ts"], 600) self.assertIsNone(args[0][0]["last_seen_ip"]) def test_connections(self) -> None: diff --git a/tests/handlers/test_device.py b/tests/handlers/test_device.py index ce7525e29c..a456bffd63 100644 --- a/tests/handlers/test_device.py +++ b/tests/handlers/test_device.py @@ -115,7 +115,7 @@ class DeviceTestCase(unittest.HomeserverTestCase): "device_id": "xyz", "display_name": "display 0", "last_seen_ip": None, - "last_seen_ts": None, + "last_seen_ts": 1000000, }, device_map["xyz"], ) |