Merge pull request #1157 from rcpch/mbarton/logout-signal-without-user

Don't crash on user_logged_out without user
rcpch · Jan 9, 2025 · 0f92703 · 0f92703
2 parents dc51f94 + 0f8463a
commit 0f92703
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/epilepsy12/signals.py b/epilepsy12/signals.py
@@ -56,10 +56,13 @@ def log_user_login_failed(sender, request, user=None, **kwargs):
 
 @receiver(user_logged_out)
 def log_user_logout(sender, request, user, **kwargs):
-    logger.info(f"{user} ({user.email}) logged out from {get_client_ip(request)}.")
-    VisitActivity.objects.create(
-        activity=3, ip_address=get_client_ip(request), epilepsy12user=user
-    )
+    if user:
+        logger.info(f"{user} ({user.email}) logged out from {get_client_ip(request)}.")
+        VisitActivity.objects.create(
+            activity=3, ip_address=get_client_ip(request), epilepsy12user=user
+        )
+    else:
+        logger.info(f"Anonymous user logged out from {get_client_ip(request)}.")
 
 
 # Epilepsy12User Signal handlers