From 14f2a9ecdec9e318d1884a6425195436d1d1d377 Mon Sep 17 00:00:00 2001 From: Mark Story Date: Wed, 3 Jul 2019 17:02:18 -0400 Subject: [PATCH] ref: Improve repr of User Having the user's id in the repr will help diagnosing which user is not being serialized properly and is causing API requests to fail. Refs SENTRY-B2M --- src/sentry/models/user.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/src/sentry/models/user.py b/src/sentry/models/user.py index 19fc99054642..6dc968d6552e 100644 --- a/src/sentry/models/user.py +++ b/src/sentry/models/user.py @@ -17,7 +17,12 @@ from django.utils import timezone from django.utils.translation import ugettext_lazy as _ -from sentry.db.models import BaseManager, BaseModel, BoundedAutoField +from sentry.db.models import ( + BaseManager, + BaseModel, + BoundedAutoField, + sane_repr +) from sentry.models import LostPasswordHash from sentry.utils.http import absolute_uri @@ -136,6 +141,8 @@ class Meta: verbose_name = _('user') verbose_name_plural = _('users') + __repr__ = sane_repr('id') + def delete(self): if self.username == 'sentry': raise Exception('You cannot delete the "sentry" user as it is required by Sentry.')