diff options
author | Erik Johnston <erik@matrix.org> | 2017-06-21 12:00:41 +0100 |
---|---|---|
committer | Erik Johnston <erik@matrix.org> | 2017-06-21 12:00:41 +0100 |
commit | 1217c7da911fe534b7804088bedc375854828ad1 (patch) | |
tree | f561943670605e96c2d47dbe9818c3bc1582e616 /synapse | |
parent | Merge pull request #2292 from matrix-org/erikj/quarantine_media (diff) | |
download | synapse-1217c7da911fe534b7804088bedc375854828ad1.tar.xz |
Don't work out users who share room with appservice users
Diffstat (limited to 'synapse')
-rw-r--r-- | synapse/handlers/user_directory.py | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/synapse/handlers/user_directory.py b/synapse/handlers/user_directory.py index d33a20a1f2..c7a3acafcb 100644 --- a/synapse/handlers/user_directory.py +++ b/synapse/handlers/user_directory.py @@ -205,6 +205,10 @@ class UserDirectoyHandler(object): count += 1 continue + if self.store.get_if_app_services_interested_in_user(user_id): + count += 1 + continue + for other_user_id in user_ids: if user_id == other_user_id: continue @@ -411,8 +415,10 @@ class UserDirectoyHandler(object): to_insert = set() to_update = set() + is_appservice = self.store.get_if_app_services_interested_in_user(user_id) + # First, if they're our user then we need to update for every user - if self.is_mine_id(user_id): + if self.is_mine_id(user_id) and not is_appservice: # Returns a map of other_user_id -> shared_private. We only need # to update mappings if for users that either don't share a room # already (aren't in the map) or, if the room is private, those that @@ -443,7 +449,10 @@ class UserDirectoyHandler(object): if user_id == other_user_id: continue - if self.is_mine_id(other_user_id): + is_appservice = self.store.get_if_app_services_interested_in_user( + other_user_id + ) + if self.is_mine_id(other_user_id) and not is_appservice: shared_is_private = yield self.store.get_if_users_share_a_room( other_user_id, user_id, ) |