Development #33056
ne pas logguer les erreurs "normales" sur api entreprises
Début:
14 mai 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
On va (souvent) interroger API Entreprise pour savoir par exemple si un Waldec ou un SIRET/SIREN existe, et donc il ne faut pas logguer les 4xx retournées par cette API
Date: Mon, 13 May 2019 17:41:05 +0200 From: root@passerelle.node2.test.saas.entrouvert.org To: admin+passerelle.test@entrouvert.com Subject: [passerelle-arles.test.entrouvert.org] ERROR (EXTERNAL IP): GET https://entreprise.api.gouv.fr/v2/associations/None/ (=> 422) GET https://entreprise.api.gouv.fr/v2/associations/None/ (=> 422) Report at /api-entreprise/api-entreprise/associations/None/ GET https://entreprise.api.gouv.fr/v2/associations/None/ (=> 422) ...
Fichiers
Révisions associées
api_entreprise: uses temporary to fix indentation (#33056)
api_entreprise: uses exception_to_text() to display exceptions (#33056)
api_entreprise: enforces PEP8 (#33056)
api_entreprise: prevents requests to log HTTP errors (#33056)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0003-api_entreprise-uses-exception_to_text-to-display-exc.patch 0003-api_entreprise-uses-exception_to_text-to-display-exc.patch ajouté
- Fichier 0005-api_entreprise-prevents-requests-to-log-HTTP-errors-.patch 0005-api_entreprise-prevents-requests-to-log-HTTP-errors-.patch ajouté
- Fichier 0002-api_entreprise-uses-temporary-to-fix-indentation-330.patch 0002-api_entreprise-uses-temporary-to-fix-indentation-330.patch ajouté
- Fichier 0001-api_entreprise-removes-dead-import-33056.patch 0001-api_entreprise-removes-dead-import-33056.patch ajouté
- Fichier 0004-api_entreprise-enforces-PEP8-33056.patch 0004-api_entreprise-enforces-PEP8-33056.patch ajouté
- Tracker changé de Support à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Seul le dernier patch est vraiment utile au ticket mais bon en passant je nettoye.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
Benjamin Dauvergne a écrit :
Seul le dernier patch est vraiment utile au ticket mais bon en passant je nettoye.
Et merci #31114.
Mis à jour par Serghei Mihai il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 6d844e152972c151bb58d95b93240b560c2a67ee Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue May 14 10:41:43 2019 +0200 api_entreprise: prevents requests to log HTTP errors (#33056) commit a2eccafe010ffd589cb75f2208d7da6451bbaa11 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue May 14 10:35:21 2019 +0200 api_entreprise: enforces PEP8 (#33056) commit d18fa62ba64588965c14e467f350fbd5b4460463 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue May 14 10:34:41 2019 +0200 api_entreprise: uses exception_to_text() to display exceptions (#33056) commit b30af7aeb7bb47cba5a9c2849e9911d2c42a4056 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue May 14 10:34:07 2019 +0200 api_entreprise: uses temporary to fix indentation (#33056) commit 247149b2f3764c24064c9da1ab0ef0218c9b49fb Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue May 14 10:33:44 2019 +0200 api_entreprise: removes dead import (#33056)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api_entreprise: removes dead import (#33056)