Skip to content

Commit

Permalink
Ensure symphony is locking room mapper in get_room_name
Browse files Browse the repository at this point in the history
Signed-off-by: Tim Paine <timothy.paine@cubistsystematic.com>
  • Loading branch information
timkpaine committed Feb 22, 2024
1 parent 6cc30bd commit 87c799f
Showing 1 changed file with 8 additions and 7 deletions.
15 changes: 8 additions & 7 deletions csp/adapters/symphony.py
Original file line number Diff line number Diff line change
Expand Up @@ -192,13 +192,14 @@ def get_room_id(self, room_name):
return room_id

def get_room_name(self, room_id):
if room_id in self._id_to_name:
return self._id_to_name[room_id]
else:
room_name = _get_room_name(room_id, self._room_info_url, self._header)
self._name_to_id[room_name] = room_id
self._id_to_name[room_id] = room_name
return room_name
with self._lock:
if room_id in self._id_to_name:
return self._id_to_name[room_id]
else:
room_name = _get_room_name(room_id, self._room_info_url, self._header)
self._name_to_id[room_name] = room_id
self._id_to_name[room_id] = room_name
return room_name

def set_im_id(self, user, id):
with self._lock:
Expand Down

0 comments on commit 87c799f

Please sign in to comment.