getsentry / sentry-ruby

Sentry SDK for Ruby
https://sentry.io/for/ruby
MIT License
933 stars 494 forks source link

Rails: ActiveSupport notification subscriptions should be spans #2398

Open nateberkopec opened 1 month ago

nateberkopec commented 1 month ago

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.

frederikspang commented 1 month ago

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

sl0thentr0py commented 1 month ago

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