Projet

Général

Profil

Development #8756

code d'erreur "personnalisable" dans les retours json

Ajouté par Serghei Mihai (congés, retour 15/05) il y a plus de 8 ans. Mis à jour il y a environ 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Serghei Mihai (congés, retour 15/05)
Version cible:
-
Début:
22 octobre 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Actuellement, en cas d'exception, jsontoresponse envoie le code d'erreur 1.
Il serait utile de pouvoir modifier ce code en fonction d'un attribut de l'exception

Fichiers

0001-allow-custom-error-codes-in-json-responses-8756.patch (979 octets) 0001-allow-custom-error-codes-in-json-responses-8756.patch Serghei Mihai (congés, retour 15/05), 22 octobre 2015 12:02
0001-return-exception-custom-error-code-in-json-responses.patch (866 octets) 0001-return-exception-custom-error-code-in-json-responses.patch Serghei Mihai (congés, retour 15/05), 22 octobre 2015 14:22
0001-to_json-can-use-err-and-HTTP-status-from-Exception-a.patch (1,08 ko) 0001-to_json-can-use-err-and-HTTP-status-from-Exception-a.patch Thomas Noël, 23 octobre 2015 00:49

Révisions associées

Révision 24499d6f (diff)
Ajouté par Thomas Noël il y a plus de 8 ans

to_json: use err and HTTP status from Exception attributes (#8756)

Révision 0c7b4e2e (diff)
Ajouté par Thomas Noël il y a plus de 8 ans

fake_family: use Exception attributes handling provided by #8756

Historique

#1

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 8 ans

  • Fichier 0001-allow-custom-error-codes-in-json-responses-8756.patch ajouté
  • Statut changé de Nouveau à En cours
  • Patch proposed changé de Non à Oui

On pourrait balancer le code d'erreur à transmettre dans le json dans le deuxième argument de l'exception

#2

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a plus de 8 ans

  • Fichier 0001-allow-custom-error-codes-in-json-responses-8756.patch supprimé
#4

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

Non, on veut avant tout fonctionner avec des exceptions bien définies.

class BlockedAccount(Exception):
     err_code = 2
#6

Mis à jour par Thomas Noël il y a plus de 8 ans

Moi j'aurai plutôt vu ça comme ça (en bonus la possibilité donnée à l'exception de spécifier le statut HTTP)

Genre

class BlockedAccount(Exception):
     err_code = 100
     http_status = 403

A noter : il ne faudra pas utiliser err_code=1 qui doit rester le code pour dire "y'a eu un problème mais je sais pas bien lequel". Je propose d'utiliser des err_code >= 100 pour nos exceptions spécifiques.

#8

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de En cours à Résolu (à déployer)
commit 24499d6fb73bd99948e2ef6ec146093a11baab3c
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Oct 23 00:44:18 2015 +0200

    to_json: use err and HTTP status from Exception attributes (#8756)
#9

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF