getsentry / sentry

Developer-first error tracking and performance monitoring
https://sentry.io
Other
37.46k stars 4.02k forks source link

Fingerprint rules not following Discover syntax #57008

Open julianCast opened 9 months ago

julianCast commented 9 months ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

Using this syntax in Discover works:

(message:"insertBefore" OR message:"removeChild" OR message:"object can not be found") and tags[boundary_translation_error]:True 
Screenshot 2023-09-27 at 10 59 15

But I'm unable to use it as fingerprint rule:

Screenshot 2023-09-27 at 10 59 46

According to the Fingerprint Documentation should be posible:

The syntax follows the syntax from Discover queries. If you want to negate the match, prefix the expression with an exclamation mark (!).

What am I doing wrong? Thank you!

Expected Result

Not syntax rule errors.

Actual Result

Syntax rule errors.

Product Area

Settings - Projects

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 9 months ago

Assigning to @getsentry/support for routing ⏲️

getsantry[bot] commented 9 months ago

Routing to @getsentry/product-owners-settings-projects for triage ⏲️

getsantry[bot] commented 9 months ago

Routing to @getsentry/product-owners-settings-general for triage ⏲️

scefali commented 9 months ago

I'm having some folks take a look, at first glance says this might be hard or impossible to do

julianCast commented 9 months ago

I'm having some folks take a look, at first glance says this might be hard or impossible to do

Thanks! The only way I've figured out how to do it is from my codebase (adding the fingerprint conditionally using Sentry library API). It's fine, I was just wondering if everything could be handled from the Sentry admin panel, but as you said, I don't think so.

So: Basic ruling => Sentry panel Conditional ruling => Sentry lib API

jjbayer commented 9 months ago

Good catch, proposed to update the docs to clarify that the syntax is different: https://github.com/getsentry/sentry-docs/pull/8094