ref(inbound-filters): Move desc and label to frontend + add link to health check doc#54846
Conversation
RaduW
left a comment
There was a problem hiding this comment.
Are we missing some of the filters (maybe the custom filters) ?
On server side we emit up to 12 filters.
Here's a list:
IP_ADDRESS = "ip-address"
RELEASE_VERSION = "release-version"
ERROR_MESSAGE = "error-message"
BROWSER_EXTENSION = "browser-extensions"
LEGACY_BROWSER = "legacy-browsers"
LOCALHOST = "localhost"
WEB_CRAWLER = "web-crawlers"
INVALID_CSP = "invalid-csp"
CORS = "cors"
DISCARDED_HASH = "discarded-hash" # Not replicated in Relay
CRASH_REPORT_LIMIT = "crash-report-limit" # Not replicated in Relay
HEALTH_CHECK = "filtered-transaction" # Ignore health-check transactions
Could we please confirm that we handle all inbound filter properly (common filters or custom filters) ?
RaduW
left a comment
There was a problem hiding this comment.
Are we missing some of the filters (maybe the custom filters) ?
On server side we emit up to 12 filters.
Here's a list:
IP_ADDRESS = "ip-address"
RELEASE_VERSION = "release-version"
ERROR_MESSAGE = "error-message"
BROWSER_EXTENSION = "browser-extensions"
LEGACY_BROWSER = "legacy-browsers"
LOCALHOST = "localhost"
WEB_CRAWLER = "web-crawlers"
INVALID_CSP = "invalid-csp"
CORS = "cors"
DISCARDED_HASH = "discarded-hash" # Not replicated in Relay
CRASH_REPORT_LIMIT = "crash-report-limit" # Not replicated in Relay
HEALTH_CHECK = "filtered-transaction" # Ignore health-check transactions
Could we please confirm that we handle all inbound filter properly (common filters or custom filters) ?
|
@RaduW the other filters come from the project settings and not from the
perhaps we should unify them one day but until then 🤷♀️ |
explained that other filters are coming from elsewhere
closes: #54539
Requires: getsentry/sentry-docs#7649
Preview: