Skip to content

Commit 14265da

Browse files
single query
1 parent da7b989 commit 14265da

File tree

1 file changed

+9
-6
lines changed
  • src/sentry/services/hybrid_cloud/user

1 file changed

+9
-6
lines changed

src/sentry/services/hybrid_cloud/user/impl.py

+9-6
Original file line numberDiff line numberDiff line change
@@ -197,13 +197,16 @@ def get_or_create_user_by_email(
197197
if user_query.count() > 1:
198198
logger.warning("Email has multiple users", extra={"email": email})
199199
if ident:
200-
for candid_user in user_query:
201-
identity_query = AuthIdentity.objects.filter(
202-
user=candid_user, ident=ident
200+
identity_query = AuthIdentity.objects.filter(
201+
user__in=user_query, ident=ident
202+
)
203+
if identity_query.exists():
204+
user = identity_query[0].user
205+
if identity_query.count() > 1:
206+
logger.warning(
207+
"Email has two auth identity for the same ident",
208+
extra={"email": email},
203209
)
204-
if identity_query.exists():
205-
user = candid_user
206-
break
207210

208211
return serialize_rpc_user(user)
209212

0 commit comments

Comments
 (0)