Bug #66560
crash 500 AssertionError sur appel à /jump/trigger/ sans préciser le nom du trigger
Début:
23 juin 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
A priori ça arrive si le statut de la demande dispose d'un saut automatique, avec ou sans trigger, sinon on a bien une 404.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 2 ans
Déjà, le test qui montre le pépin, à corriger, en mode Tears-driven development :
diff --git a/tests/api/test_workflow.py b/tests/api/test_workflow.py index bcb55ef59..6d0e593be 100644 --- a/tests/api/test_workflow.py +++ b/tests/api/test_workflow.py @@ -97,6 +97,12 @@ def test_workflow_trigger(pub, local_user): formdata.store() assert formdef.data_class().get(formdata.id).status == 'wf-st1' + # incomplete URL + get_app(pub).post(sign_uri(formdata.get_url() + 'jump/'), status=404) <-- ok + get_app(pub).post(sign_uri(formdata.get_url() + 'jump'), status=404) <-- ok + get_app(pub).post(sign_uri(formdata.get_url() + 'jump/trigger/'), status=404) <-- crash AssertionError ici + get_app(pub).post(sign_uri(formdata.get_url() + 'jump/trigger'), status=404) + get_app(pub).post(sign_uri(formdata.get_url() + 'jump/trigger/XXX'), status=200) assert formdef.data_class().get(formdata.id).status == 'wf-st2' assert formdef.data_class().get(formdata.id).evolution[-1].who is None
Mis à jour par Thomas Noël il y a presque 2 ans
- Fichier 0001-jump-raise-404-if-trigger-name-is-missing-66560.patch 0001-jump-raise-404-if-trigger-name-is-missing-66560.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Thomas Noël
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 2ea9ad0ee81aa07136ec1f494a998fb7ba0ef7fc Author: Thomas NOEL <tnoel@entrouvert.com> Date: Thu Jun 23 17:00:08 2022 +0200 jump: raise 404 if trigger name is missing (#66560)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
jump: raise 404 if trigger name is missing (#66560)