getsentry / getsentry/sentry

Send alert ID in webhook request body

Open
#70,742 3 comments 1 reaction 0 assignees View on GitHub
Feature Product Area: Settings - Integrations
Dominant language
Python
Stars
44.8k
Forks
4.9k
Avg merge
22h 21m
Merged PRs (30d)
586

Description

### Problem Statement

Currently, the request body sent in a webhook triggered by a sentry alert only includes the name of the triggered rule - this is too loose, and if someone else updates the Sentry configuration and just (for example) changes the casing in the alert's name, this'll break any code that needs to perform specific actions based on a trigger

For instance, we have a few specific alerts to disable services in the web app based on high occurances of errors in them (i.e. more than 10 exceptions thrown that are tagged with 'sms' in more than a minute triggers a webhook which, when handled by our server, disables the sms service)

### Solution Brainstorm

Include the alert ID in the webhook body

### Product Area

Unknown

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.