element-hq / element-hq/dendrite

Update Sentry reporting further

Open
#3,306 0 comments 0 reactions 0 assignees View on GitHub
good first issue T-Task
Dominant language
Go
Stars
965
Forks
101
PR merge metrics
No merged PRs in 30d

Description

*This issue was originally created by [**@S7evinK**](https://github.com/S7evinK) at .*

https://github.com/matrix-org/dendrite/pull/3305 made reporting errors a bit better, but there are still places in the code where we `CaptureException` in go routines, possibly still sending garbage to Sentry.
For reference: https://docs.sentry.io/platforms/go/usage/concurrency/

It may also make sense to check if we even should report said errors to sentry, as e.g. we should always be able `json.Unmarshal` things we sent to JetStream ourselves.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.