Skip to content

Commit f32c95c

Browse files
authored
Fix app request user bug (#28)
1 parent c5f0a3a commit f32c95c

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

apps/handlers/slack_app.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,8 @@
1717
from base.models import Profile
1818
from processors.providers.slack.post_message import SlackPostMessageProcessor
1919

20+
from django.contrib.auth.models import User, AnonymousUser
21+
2022
logger = logging.getLogger(__name__)
2123

2224

@@ -61,15 +63,12 @@ def _get_app_request_user(self, slack_request_payload):
6163
)
6264
if slack_user_email is not None:
6365
self._slack_user_email = slack_user_email
64-
profile_object = Profile.objects.get(
65-
user__email=slack_user_email,
66-
)
67-
if profile_object is not None:
68-
return profile_object.user
66+
user_object = User.objects.get(email=slack_user_email)
67+
return user_object if user_object is not None else AnonymousUser()
6968
except Exception as e:
70-
pass
69+
logger.exception(f"Error in fetching user object from slack payload {slack_request_payload}")
7170

72-
return None
71+
return AnonymousUser()
7372

7473
def _get_slack_app_seession_id(self, slack_request_payload):
7574
if slack_request_payload['type'] == 'event_callback' and 'event' in slack_request_payload:

0 commit comments

Comments
 (0)