fix: Add user id in django integration

untitaker · untitaker · commit c1e0ba03b343 · 2018-11-26T21:20:21.000+01:00
diff --git a/sentry_sdk/integrations/django/__init__.py b/sentry_sdk/integrations/django/__init__.py
@@ -189,6 +189,12 @@ def _set_user_info(request, event):
     if user is None or not is_authenticated(user):
         return
 
+    if "id" not in user_info:
+        try:
+            user_info["id"] = str(user.pk)
+        except Exception:
+            pass
+
     if "email" not in user_info:
         try:
             user_info["email"] = user.email
diff --git a/tests/integrations/django/test_basic.py b/tests/integrations/django/test_basic.py
@@ -93,7 +93,11 @@ def test_user_captured(sentry_init, client, capture_events):
 
     event, = events
 
-    assert event["user"] == {"email": "lennon@thebeatles.com", "username": "john"}
+    assert event["user"] == {
+        "email": "lennon@thebeatles.com",
+        "username": "john",
+        "id": "1",
+    }
 
 
 def test_404(sentry_init, client):
