Projet

Général

Profil

Development #49167

Sentry - grouper les issues

Ajouté par Lauréline Guérin il y a plus de 3 ans. Mis à jour il y a plus de 3 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
07 décembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

On a beaucoup d'issues genre https://sentry.entrouvert.org/entrouvert/publik/issues/17952/ avec le message qui diffère légèrement

"GET https://api-adresse.data.gouv.fr/search/<params> (=> ReadTimeout(ReadTimeoutError("HTTPSConnectionPool(host='api-adresse.data.gouv.fr', port=443): Read timed out. (read timeout=25)")))"

On pourrait les grouper pour y voir plus clair dans sentry.

https://docs.sentry.io/product/sentry-basics/guides/grouping-and-fingerprints/

On peut merger les issues à la main dans l'interface sentry, ou changer le fingerprint au niveau du code:
https://docs.sentry.io/platforms/python/data-management/event-grouping/sdk-fingerprinting/

Ca nécessite d'ajouter une dépendance à sentry dans le code (from sentry_sdk import push_scope, capture_exception etc)

Je pense qu'on pourrait intégrer ça dans passerelle.utils.jsonresponse (dans la méthode api, où on appelle log_http_request qui produit cette trace)

Vous en pensez quoi ?

Historique

#1

Mis à jour par Lauréline Guérin il y a plus de 3 ans

Note: j'ai pas trouvé dans l'interface sentry où setter les fingerprint rules, ce qui nous éviterait de toucher au code
https://docs.sentry.io/product/sentry-basics/guides/grouping-and-fingerprints/#fingerprint-rules

#4

Mis à jour par Benjamin Dauvergne il y a plus de 3 ans

Concernant le fait de le faire dans sentry, je recopie ce que j'ai dit dans l'autre ticket, ce n'est possible qu'avec la version payante proprio.

#5

Mis à jour par Lauréline Guérin il y a plus de 3 ans

  • Statut changé de Nouveau à Rejeté

Formats disponibles : Atom PDF