Skip to content

Commit f7d53d7

Browse files
authored
chore(alerts): Stop sending unused count headers from combined rules endpoint (#74613)
These headers don't look to be used in our codebase and are undocumented. They're likely contributing to slowness on this endpoint, so just removing.
1 parent 9db72d0 commit f7d53d7

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

src/sentry/incidents/endpoints/organization_alert_rule_index.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -285,8 +285,6 @@ def get(self, request: Request, organization) -> Response:
285285
)
286286
issue_rules = issue_rules.annotate(date_triggered=far_past_date)
287287
uptime_rules = uptime_rules.annotate(date_triggered=far_past_date)
288-
alert_rules_count = alert_rules.count()
289-
issue_rules_count = issue_rules.count()
290288
alert_rule_intermediary = CombinedQuerysetIntermediary(alert_rules, sort_key)
291289
rule_intermediary = CombinedQuerysetIntermediary(issue_rules, rule_sort_key)
292290
uptime_intermediary = CombinedQuerysetIntermediary(uptime_rules, rule_sort_key)
@@ -300,8 +298,6 @@ def get(self, request: Request, organization) -> Response:
300298
cursor_cls=StringCursor if case_insensitive else Cursor,
301299
case_insensitive=case_insensitive,
302300
)
303-
response["X-Sentry-Issue-Rule-Hits"] = issue_rules_count
304-
response[ALERT_RULES_COUNT_HEADER] = alert_rules_count
305301
response[MAX_QUERY_SUBSCRIPTIONS_HEADER] = settings.MAX_QUERY_SUBSCRIPTIONS_PER_ORG
306302
return response
307303

0 commit comments

Comments
 (0)