Projet

Général

Profil

Bug #17826

permettre un log_errors=False lors des requêtes json cell

Ajouté par Thomas Noël il y a presque 7 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
28 juillet 2017
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Actuellement toutes les requêtes json_cells font une erreur (trace) en cas de réponse non 200, ce n'est pas toujours voulu (genre 404 peut être une information).

Comme on renvoie maintenant json_status, on peut autoriser de faire du log_errors=False


Fichiers

Révisions associées

Révision c6cf29c9 (diff)
Ajouté par Thomas Noël il y a presque 7 ans

json cell: add log_errors parameter (#17826)

Historique

#1

Mis à jour par Thomas Noël il y a presque 7 ans

#2

Mis à jour par Frédéric Péters il y a presque 7 ans

+    log_errors = models.BooleanField(_('Log request errors'),
+            default=JsonCellBase.log_errors)

Manque dès lors la migration. Mais est-ce un paramètre qu'on veut exposer là ? (pas vraiment d'opinion)

#3

Mis à jour par Thomas Noël il y a presque 7 ans

Frédéric Péters a écrit :

Manque dès lors la migration. Mais est-ce un paramètre qu'on veut exposer là ? (pas vraiment d'opinion)

C'est vrai que c'est le mode prototype et que ça doit vraiment le rester, "embêtons" ceux qui veulent l'utiliser en vrai. Et une migration en moins, c'est bien.

#4

Mis à jour par Frédéric Péters il y a presque 7 ans

Ok pour moi ainsi.

#5

Mis à jour par Thomas Noël il y a presque 7 ans

  • Statut changé de En cours à Résolu (à déployer)
commit c6cf29c988d2cea3c6a81d4199303d2e99743e7e
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Jul 28 00:42:18 2017 +0200

    json cell: add log_errors parameter (#17826)

#6

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

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

Formats disponibles : Atom PDF