Bug #62398
Astre REST : getRefEntite ne renvoie pas du json (et c'est normal, c'est documenté)
Début:
03 mars 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Non
Planning:
Non
Description
Un coup json, un coup pas json, pratique.
Fichiers
Révisions associées
Historique
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Fichier 0001-astre_rest-handle-non-json-response-in-gf-documents-.patch 0001-astre_rest-handle-non-json-response-in-gf-documents-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
C'est toi que ça regarde, mais j'aurais plutôt appelé l'argument « json_response=True » histoire de pas confuser un jour avec un paramètre qui ressemble au json de requests.
Une autre solution plus brutale serait de juste faire un try/except autour de return resp.json() :
try: return resp.json() except ValueError, TypeError: return resp.text
Sale mais efficace.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de Solution validée à En cours
- Patch proposed changé de Oui à Non
En fait non je vais tenter le json, attraper l'exception et renvoyer alors le contenu brut plutôt que gérer ça via un paramètre.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
Ah les grands esprits se rencontrent, je garde ta validation du coup, je pousserai direct.
Mis à jour par Emmanuel Cazenave il y a environ 2 ans
- Statut changé de En cours à Résolu (à déployer)
commit 972a4f885f0c02a4f93da22feb17750b1bbe973a Author: Emmanuel Cazenave <ecazenave@entrouvert.com> Date: Thu Mar 3 18:00:38 2022 +0100 astre_rest : handle non json response (#62398)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
astre_rest : handle non json response (#62398)