Development #17168
json data store : content doit être un dict
Début:
24 juin 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
et donc tester les json.loads(request.body)
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Fichier 0001-jsondatastore-check-if-request.body-is-a-json-17168.patch 0001-jsondatastore-check-if-request.body-is-a-json-17168.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Thomas Noël tu pensais à un truc comme ça ?
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Je pense qu'il attendait en plus un if not isinstance(content, dict): raise APIError('un dict on a dit!!!')
.
Mis à jour par Lauréline Guérin il y a plus de 4 ans
Mis à jour par Lauréline Guérin il y a plus de 4 ans
J'ai essayé de reprendre les messages d'erreur qu'on peut trouver ailleurs dans le projet, pour un soucis d'homogénéité.
J'ai préféré ajouter mes checks dans un test existant, pour éviter de faire trop différent de ce qui existe déjà, mais si vous préférez que j'isole les tests dites-moi :)
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 96394aca7c0719af2b4154fd108837a0bd55283c Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Wed Oct 2 17:03:44 2019 +0200 jsondatastore: check if request.body is a json (#17168)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
jsondatastore: check if request.body is a json (#17168)