This repository has been archived by the owner on Nov 25, 2024. It is now read-only.
Update Sentry reporting further #3306
Labels
good first issue
Want to help with Dendrite? These are the issues to start with!
T-Task
Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
#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.The text was updated successfully, but these errors were encountered: