Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #26801 from edx/ddumesnil/aa-513-user-id-fix

Browse Source 
AA-513: Ensure user id exists before trying to send event
This commit is contained in:
Dillon Dumesnil
2021-03-02 06:55:52 -08:00
committed by GitHub
parent 53d9f66217 aeb71250e9
commit 49b67d5a99
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
common/djangoapps/student/models.py
View File

@@ -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)