From 1a54414afff62561dff03e8a9f43287fa38176b6 Mon Sep 17 00:00:00 2001 From: jason Date: Mon, 1 Aug 2016 11:34:14 -0600 Subject: Fix a bug where sometimes the nick isn't in the change user info --- warmachine/connections/slack.py | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/warmachine/connections/slack.py b/warmachine/connections/slack.py index 90903e7..7f54a55 100644 --- a/warmachine/connections/slack.py +++ b/warmachine/connections/slack.py @@ -166,12 +166,17 @@ class SlackWS(Connection): https://api.slack.com/events/user_change """ user_info = msg['user'] - old_nick = self.user_map[user_info['id']]['nick'] + try: + old_nick = self.user_map[user_info['id']]['nick'] + except KeyError as e: + old_nick = None + self.log.exception('KeyError: {}'.format(e)) + self.log.exception('{}'.format(msg)) self.user_map[user_info['id']] = user_info # Update the nick mapping if the user changed their nickname - if old_nick != user_info['nick']: + if old_nick and old_nick != user_info['nick']: del self.user_nick_to_id[old_nick] self.user_nick_to_id[user_info['nick']] = user_info['id'] -- cgit v1.2.1