Skip to content

Commit 2e20d1c

Browse files
authored
ref(post-process): remove unecessary try catch (#93859)
all post process jobs are already wrapped in try catch, so one inside this function is redundant. https://github.com/getsentry/sentry/blob/31b8a0c441d69fd8a9dd57c13b2e7f63a2a082df/src/sentry/tasks/post_process.py#L681
1 parent 559ec15 commit 2e20d1c

File tree

1 file changed

+12
-15
lines changed

1 file changed

+12
-15
lines changed

src/sentry/tasks/post_process.py

Lines changed: 12 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1168,21 +1168,18 @@ def handle_auto_assignment(job: PostProcessJob) -> None:
11681168
from sentry.models.projectownership import ProjectOwnership
11691169

11701170
event = job["event"]
1171-
try:
1172-
ProjectOwnership.handle_auto_assignment(
1173-
project_id=event.project_id,
1174-
organization_id=event.project.organization_id,
1175-
event=event,
1176-
logging_extra={
1177-
"event_id": event.event_id,
1178-
"group_id": str(event.group_id),
1179-
"project_id": str(event.project_id),
1180-
"organization_id": event.project.organization_id,
1181-
"source": "post_process",
1182-
},
1183-
)
1184-
except Exception:
1185-
logger.exception("Failed to set auto-assignment")
1171+
ProjectOwnership.handle_auto_assignment(
1172+
project_id=event.project_id,
1173+
organization_id=event.project.organization_id,
1174+
event=event,
1175+
logging_extra={
1176+
"event_id": event.event_id,
1177+
"group_id": str(event.group_id),
1178+
"project_id": str(event.project_id),
1179+
"organization_id": event.project.organization_id,
1180+
"source": "post_process",
1181+
},
1182+
)
11861183

11871184

11881185
def process_service_hooks(job: PostProcessJob) -> None:

0 commit comments

Comments
 (0)