Update sentry reporting

[S7evinK]

This hopefully reduces the garbage we currently produce.
(Using GlitchTip on my personal instance, this seems to look better)

[codecov]

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (8e4dc6b) 65.53% compared to head (fa1f1f3) 65.52%.

Files	Patch %	Lines
federationapi/routing/routing.go	0.00%	2 Missing ⚠️
internal/httputil/httpapi.go	0.00%	2 Missing ⚠️
relayapi/routing/routing.go	0.00%	2 Missing ⚠️
roomserver/internal/input/input_latest_events.go	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3305      +/-   ##
==========================================
- Coverage   65.53%   65.52%   -0.01%     
==========================================
  Files         509      509              
  Lines       57514    57522       +8     
==========================================
  Hits        37694    37694              
- Misses      15951    15959       +8     
  Partials     3869     3869              

Flag	Coverage Δ
unittests	50.11% <53.33%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[devonh]

Wouldn't we also want to do this here as well:

dendrite/federationapi/routing/routing.go

Line 648 in bebf701

hub := sentry.GetHubFromContext(req.Context())

[devonh]

Should we be doing a similar thing everywhere we use CaptureException or CaptureMessage?
The docs would suggest we should (at least everywhere being run inside a goroutine) https://docs.sentry.io/platforms/go/usage/concurrency/

[S7evinK]

Created #3306 as a follow up task