Project

General

Profile

Development #84348

cmis: ne pas envoyer les erreurs HTTP à sentry

Added by Benjamin Dauvergne 3 months ago. Updated 2 months ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
-
Start date:
06 December 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

Description

La plupart des erreurs HTTP sont internes au fonctionnement de CMIS (404 chemin inexistant, 403 erreur d'accès et 409 chemin qui existe déjà) il n'est pas nécessaire de les remonter dans sentry, de plus elles sont visibles dans les logs et c'est suffisant.

Un dernier exemple https://sentry.entrouvert.org/entrouvert/publik/issues/105637/.

Associated revisions

Revision f15d802b (diff)
Added by Benjamin Dauvergne 3 months ago

cmis: improve reporting of ObjectNotfound and PermissionDenied (#84348)

Revision b4d63724 (diff)
Added by Benjamin Dauvergne 3 months ago

cmis: disable logging of requests errors (#84348)

History

#1

Updated by Benjamin Dauvergne 3 months ago

  • Tracker changed from Support to Development
#2

Updated by Robot Gitea 3 months ago

  • Status changed from Nouveau to Solution proposée
  • Assignee set to Benjamin Dauvergne

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#3

Updated by Benjamin Dauvergne 3 months ago

  • Subject changed from cmis: ne pas logger les erreurs HTTP to cmis: ne pas envoyer les erreurs HTTP à sentry
  • Status changed from Solution proposée to Nouveau
  • Assignee deleted (Benjamin Dauvergne)
#4

Updated by Benjamin Dauvergne 3 months ago

  • Status changed from Nouveau to Solution proposée
#5

Updated by Robot Gitea 3 months ago

  • Status changed from Solution proposée to Solution validée

Nicolas Roche (nroche) a approuvé une pull request sur Gitea concernant cette demande :

#6

Updated by Robot Gitea 3 months ago

  • Status changed from Solution validée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#7

Updated by Transition automatique 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#8

Updated by Transition automatique 11 days ago

Automatic expiration

Also available in: Atom PDF