Project

General

Profile

Bug #66560

crash 500 AssertionError sur appel à /jump/trigger/ sans préciser le nom du trigger

Added by Thomas Noël (en congés → 18 juillet) 10 days ago. Updated 6 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
23 Jun 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

Associated revisions

Revision 2ea9ad0e (diff)
Added by Thomas Noël (en congés → 18 juillet) 9 days ago

jump: raise 404 if trigger name is missing (#66560)

History

#2

Updated by Thomas Noël (en congés → 18 juillet) 10 days ago

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
#3

Updated by Thomas Noël (en congés → 18 juillet) 10 days ago

#4

Updated by Lauréline Guerin 9 days ago

  • Status changed from Solution proposée to Solution validée
  • Assignee set to Thomas Noël (en congés → 18 juillet)
#5

Updated by Thomas Noël (en congés → 18 juillet) 9 days ago

  • Status changed from Solution validée to 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)

#6

Updated by Transition automatique 6 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF