Projet

Général

Profil

Development #35358

arpege_ecp, ne pas logger les erreurs de requêtes HTTPs

Ajouté par Benjamin Dauvergne il y a plus de 4 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Benjamin Dauvergne
Version cible:
-
Début:
11 août 2019
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Parce que ça vient d'Arpège et que personne ne corrigera.

Fichiers

0001-arpege_ecp-do-not-log-requests-errors-35358.patch (3,75 ko) 0001-arpege_ecp-do-not-log-requests-errors-35358.patch Benjamin Dauvergne, 11 août 2019 17:02
0001-arpege_ecp-do-not-log-requests-errors-35358.patch (8,98 ko) 0001-arpege_ecp-do-not-log-requests-errors-35358.patch Benjamin Dauvergne, 12 août 2019 12:26

Révisions associées

Révision 47529acd (diff)
Ajouté par Benjamin Dauvergne il y a plus de 4 ans

arpege_ecp: do not log requests errors (#35358)

Add proper conversion of all HTTP errors to APIError.

Historique

#1

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

#2

Mis à jour par Serghei Mihai il y a plus de 4 ans

A la place de:

raise Exception('Arpege server is down: %s' % e)

plutôt:

raise APIError('Arpege server is down: %s' % e)

non?

#3

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

Dans check_status ça n'a pas d'importance. Correction aux tests qui n'utilisaient pas httmock.

#4

Mis à jour par Serghei Mihai il y a plus de 4 ans

  • Statut changé de Solution proposée à Solution validée
#5

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 47529acd9a6637d4248084594634f389c18066dd
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sun Aug 11 16:54:46 2019 +0200

    arpege_ecp: do not log requests errors (#35358)

    Add proper conversion of all HTTP errors to APIError.
#6

Mis à jour par Frédéric Péters il y a plus de 4 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF