From aeb71250e9a7f9ae4e75e48daad5768d7465a3b5 Mon Sep 17 00:00:00 2001 From: Dillon Dumesnil Date: Tue, 2 Mar 2021 09:22:12 -0500 Subject: [PATCH] AA-513: Ensure user id exists before trying to send event request.user.id could be None which ends up raising an error in the analytics package we use. --- common/djangoapps/student/models.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/common/djangoapps/student/models.py b/common/djangoapps/student/models.py index f1ffc97b70..9d08e05996 100644 --- a/common/djangoapps/student/models.py +++ b/common/djangoapps/student/models.py @@ -2558,7 +2558,8 @@ def log_successful_logout(sender, request, user, **kwargs): # lint-amnesty, pyl AUDIT_LOG.info('Logout - user.id: {0}'.format(request.user.id)) # pylint: disable=logging-format-interpolation else: AUDIT_LOG.info('Logout - {0}'.format(request.user)) # pylint: disable=logging-format-interpolation - segment.track(request.user.id, 'edx.bi.user.account.logout') + if request.user.id: + segment.track(request.user.id, 'edx.bi.user.account.logout') @receiver(user_logged_in)