getsentry / sentry

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

Breadcrumbs can contain broken transaction links #49600

Closed smeubank closed 1 year ago

smeubank commented 1 year ago

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. find an issue with breadcrumbs
  2. issues should have transactions
  3. click link to go to transaction detail

Expected Result

I expect that a non-clickable link should be disabled, or that it isn't there at all,

if the ID is available at time of bread crumbs creation then potentially the transaction name is as well, which should be more useful

Potentially these could be deleted, discarded (client side sampling), or dropped (regular server side reasons) or performance at scale[dynamic sampling]

Actual Result

will take you to a broken page

Product Area

Issues

Link

No response

DSN

No response

Version

No response

getsantry[bot] commented 1 year ago

Assigning to @getsentry/support for routing, due by (sfo). ⏲️

getsantry[bot] commented 1 year ago

Routing to @getsentry/product-owners-issues for triage, due by (sfo). ⏲️