Skip to content

Commit db5cbea

Browse files
authored
chore(invite-members): add logging for reinvite (#80436)
we want to log usage of this `OrganizationMemberDetailsEndpoint` to ensure that this is not being called to reinvite AND update member details at the same time, before we limit the endpoint to only 1 action (invite or update) at a time
1 parent e2dcdc0 commit db5cbea

File tree

1 file changed

+16
-0
lines changed
  • src/sentry/api/endpoints/organization_member

1 file changed

+16
-0
lines changed

src/sentry/api/endpoints/organization_member/details.py

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
from __future__ import annotations
22

3+
import logging
4+
35
from django.db import router, transaction
46
from django.db.models import Q
57
from drf_spectacular.utils import extend_schema, inline_serializer
@@ -241,9 +243,23 @@ def put(
241243
if not is_invite_from_user:
242244
return Response({"detail": ERR_MEMBER_INVITE}, status=403)
243245

246+
logger = logging.getLogger(__name__)
247+
244248
# XXX(dcramer): if/when this expands beyond reinvite we need to check
245249
# access level
246250
if result.get("reinvite"):
251+
logger.info(
252+
"organization.member_reinvite",
253+
extra={
254+
"organization_id": organization.id,
255+
"user_id": request.user.id,
256+
"member_id": member.id,
257+
"orgRole": result.get("orgRole"),
258+
"role": result.get("role"),
259+
"teams": result.get("teams"),
260+
"teamRoles": result.get("teamRoles"),
261+
},
262+
)
247263
if member.is_pending:
248264
if ratelimits.for_organization_member_invite(
249265
organization=organization,

0 commit comments

Comments
 (0)