Projet

Général

Profil

Development #51182

ne pas remonter vers Sentry les APIError de djangorestframework (?)

Ajouté par Frédéric Péters il y a environ 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Rejeté
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
16 février 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Actuellement c'est le cas mais l'erreur doit déjà être visible pour l'appelant, on n'a pas besoin de l'avoir également du côté de l'API (ex: https://sentry.entrouvert.org/entrouvert/publik/issues/29703/ sur une erreur de format lors de l'appel à une API Chrono).

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

APIError est une exception spécifique à chrono qui ne dérive pas de la classe d'exception prévue par DRF qui est APIException qui doit être géré spécifiquement, c'est d'ailleurs comme ça que c'est fait partout dans le code de chrono pour l'instant.

Le plus simple à mon avis c'est de garder APIError tel quel mais d'ajouter un exception_handler spécifique1 dans chrono.

1 https://www.django-rest-framework.org/api-guide/exceptions/#custom-exception-handling

#2

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Projet changé de Hobo à Chrono
#3

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

  • Statut changé de Nouveau à Rejeté

fermé en faveur de #51181

Formats disponibles : Atom PDF