sentry-ruby icon indicating copy to clipboard operation
sentry-ruby copied to clipboard

Rails: ActiveSupport notification subscriptions should be spans

Open nateberkopec opened this issue 1 year ago • 2 comments

Currently, a lot of ActiveSupport notifications are used to create breadcrumbs

I think pretty much all of these would also be useful spans, particularly the cache stuff.

nateberkopec avatar Sep 10 '24 05:09 nateberkopec

Cache is being worked on in https://github.com/getsentry/sentry-ruby/pull/2380 - Not as breadcrumb :)

frederikspang avatar Sep 10 '24 15:09 frederikspang

thx @nateberkopec, we could definitely update our list to include more spans. I'm a bit wary of including everything because

  • we had serialization problems in the past with some of them
  • in other languages, we've had some spans that just act as noise so I'd like to avoid high volume stuff

sl0thentr0py avatar Sep 11 '24 11:09 sl0thentr0py