Projet

Général

Profil

Bug #12418

call_webservice() pour un POST sans payload échoue dans le calcul du timeout

Ajouté par Benjamin Dauvergne il y a presque 8 ans. Mis à jour il y a plus de 7 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
02 juillet 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers

Révisions associées

Révision 527ae5a3 (diff)
Ajouté par Benjamin Dauvergne il y a presque 8 ans

wscalls: measure payload only if there is one (#12418)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Le test échoue sans le patch.

#2

Mis à jour par Frédéric Péters il y a presque 8 ans

  • Statut changé de Nouveau à Résolu (à déployer)

J'ai ajouté la référence au ticket et poussé. Ça faisait suite à une erreur rencontrée sur une installation ?

commit 527ae5a3d86f578f7811b2c514c5166feba1e290
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Sat Jul 2 18:43:09 2016 +0200

    wscalls: measure payload only if there is one (#12418)
#3

Mis à jour par Frédéric Péters il y a presque 8 ans

Yes, chateauroux.test

REQUEST_URI     '/backoffice/management/inscription-aux-ateliers-adulte/2/'
HTTP_HOST       'demarches-chateauroux.test.entrouvert.org'
TypeError: object of type 'NoneType' has no len()
(18 additional frame(s) were not displayed)
...
  File "wcs/workflows.py", line 1369, in handle_form
    url = filled.perform_workflow()
  File "wcs/formdata.py", line 356, in perform_workflow
    url = perform_items(wf_status.items, self)
  File "wcs/workflows.py", line 60, in perform_items
    url = item.perform(formdata) or url
  File "wcs/wf/wscall.py", line 246, in perform
    formdata=formdata)
  File "wcs/wscalls.py", line 105, in call_webservice
    timeout += len(payload) / 65536
#4

Mis à jour par Frédéric Péters il y a presque 8 ans

Et j'ai donc appliqué le patch sur recette et prod.

#5

Mis à jour par Benjamin Dauvergne il y a presque 8 ans

Non je l'ai trouvé en développant le ticket sur le support PUT, PATCH, DELETE.
Mais c'est bien si ça corrige un bug connu aussi :)

#6

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Version cible mis à v1.49
#7

Mis à jour par Frédéric Péters il y a plus de 7 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF