Skip to content

Commit ea4a3cf

Browse files
use Scope.set_user instead of individual attributes
1 parent 3db9491 commit ea4a3cf

1 file changed

Lines changed: 6 additions & 12 deletions

File tree

sentry_sdk/integrations/django/__init__.py

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -482,29 +482,23 @@ def _after_get_response(request: "WSGIRequest") -> None:
482482
if user is None or not is_authenticated(user):
483483
return
484484

485-
user_id = None
485+
user_info = {}
486486
try:
487-
user_id = str(user.pk)
487+
user_info["id"] = str(user.pk)
488488
except Exception:
489489
pass
490-
if user_id is not None:
491-
scope.set_attribute(SPANDATA.USER_ID, user_id)
492490

493-
user_email = None
494491
try:
495-
user_email = user.email
492+
user_info["email"] = user.email
496493
except Exception:
497494
pass
498-
if user_email is not None:
499-
scope.set_attribute(SPANDATA.USER_EMAIL, user_email)
500495

501-
username = None
502496
try:
503-
username = user.get_username()
497+
user_info["username"] = user.get_username()
504498
except Exception:
505499
pass
506-
if username is not None:
507-
scope.set_attribute(SPANDATA.USER_NAME, username)
500+
501+
scope.set_user(user_info)
508502

509503

510504
def _patch_get_response() -> None:

0 commit comments

Comments
 (0)