matrix-org/dendrite

Update Sentry reporting further

Aperta

#3306 aperta il 18 gen 2024

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Go (658 fork)auto 404
T-Taskgood first issue

Metriche repository

Star
 (5632 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

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.

Guida contributor

Direzione di ricerca
Identifica le goroutine nel codice che chiamano sentry.CaptureException e valuta se l'errore dovrebbe essere segnalato. Controlla anche se l'errore proviene dalla deserializzazione di JetStream e, in tal caso, salta la segnalazione.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitGo
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65