matrix-org/dendrite

Update Sentry reporting further

Offen

#3.306 geöffnet am 18.01.2024

Auf GitHub ansehenContributor Guide
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Go (658 Forks)auto 404
T-Taskgood first issue

Repository-Metriken

Stars
 (5.632 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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

Research-Richtung
Identifizieren Sie Goroutinen im Code, die sentry.CaptureException aufrufen, und bewerten Sie, ob der Fehler gemeldet werden sollte. Überprüfen Sie auch, ob der Fehler von der JetStream Deserialisierung stammt, und überspringen Sie in diesem Fall die Meldung.
Tech Stack
go
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Frisch
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
GitGo
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65