Development #10853
meyzieu_newsletters: gérer les retours d'erreur via des codes HTTP
0%
Description
S'assurer que 200 est renvoyé en cas de succès(récuperation, publication des newsletters)
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 8 ans
Mon commentaire dans #10838 :
Cela étant; en cas d'erreur, le set_subscriptions des newsletter meyzieu fait un return False et le résultat ça va être une réponse {"data": false, "err": 0} et un code 200. Si je lis correctement. Et donc me soucier de la compatibilité de la gestion d'erreurs alors qu'il n'y avait rien, c'est futile.
Il ne s'agit pas d'assurer un 200 en cas de succès (c'est déjà le cas) mais bien d'assurer autre chose en cas d'erreurs (identique à ce que fait/fera corbo).
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
- Fichier 0001-meyzieu_newsletters-return-503-code-on-webservice-co.patch 0001-meyzieu_newsletters-return-503-code-on-webservice-co.patch ajouté
- Patch proposed changé de Non à Oui
Je vois 2 cas ou les erreurs doivent être gérées via des codes http: la récuperation et la mise à jour des abonnements.
Comme les erreurs peuvent survenir en cas de problème de connexion au webservice des abonnements je propose retourner un 503 (Service unavailable).
Mis à jour par Frédéric Péters il y a environ 8 ans
http_status=503 : il faut des espaces autour du =.
Aussi, on ne peut pas dépendre uniquement d'un statut HTTP, il me semble que le @to_json va également poser un contenu, err=1, err_class, err_desc, ce serait bien de s'assurer de ça en le vérifiant dans les tests.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 8 ans
- Statut changé de Nouveau à Résolu (à déployer)
- Assigné à mis à Serghei Mihai (congés, retour 15/05)
commit 8b12e5c64acb7a00df1799bb8447494b4d5aa1e4
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Fri May 6 14:31:45 2016 +0200
meyzieu_newsletters: return 503 code on webservice connection error (#10853)
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a presque 8 ans
- Statut changé de Résolu (à déployer) à Fermé
meyzieu_newsletters: return 503 code on webservice connection error (#10853)