Projet

Général

Profil

Bug #66560

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

Ajouté par Thomas Noël il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
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

Révision 2ea9ad0e (diff)
Ajouté par Thomas Noël il y a presque 2 ans

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

Historique

#2

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

Mis à jour par Thomas Noël il y a presque 2 ans

#4

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

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)

#6

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#7

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF