Bug #12381
crash appel wscall en mode cron, à cause de tentative de signature
Début:
30 juin 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
En mode cron (action de workflow lancée suite à expiration), on n'a pas de get_server, et le système de signature automagique se plante :
Subject: Quixote Traceback ([CRON] AttributeError: 'NoneType' object has no attribute 'get_server')
Exception:
type = '<type 'exceptions.AttributeError'>', value = ''NoneType' object has no attribute 'get_server''
Stack trace (most recent call first):
File "/usr/lib/python2.7/dist-packages/wcs/api_utils.py", line 134, in get_secret_and_orig
132
133 def get_secret_and_orig(url):
> 134 orig = get_request().get_server().split(':')[0]
135 target_orig = urlparse.urlparse(url).netloc.rsplit('@', 1)[-1].rsplit(':', 1)[0]
136 secret = get_publisher().get_site_option(target_orig, 'wscall-secrets')
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 8 ans
On n'a pas de get_request(), plutôt.
Je pense que ça n'amènerait pas de problème de poser un faux request sur le publisher.
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
Ne pourrait-on calculer l'origine simplement à partir du app_dir ? Ça me rappelle le même genre de problème dans les applications Django lorsqu'on utilisait les objets HttpRequest plutôt que Tenant pour calculer une base_url.
Mis à jour par Frédéric Péters il y a presque 8 ans
Ouaip, il y a divers moyens pour y arriver, ça ne posera pas de problème à faire.
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
- Fichier 0001-use-frontoffice_url-to-compute-our-orig-fixes-12381.patch 0001-use-frontoffice_url-to-compute-our-orig-fixes-12381.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 8 ans
Un test sur le côté, où il y aurait un publisher mais pas de request ?
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de Nouveau à Résolu (à déployer)
J'ai adapté le message et fait des petites modifications d'espaces et poussé.
commit 16ce4fc880a534a58ac749b704b51ddde9a67f6b
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Jun 30 15:36:23 2016 +0200
misc: use frontoffice_url to compute orig parameter (#12381)
misc: use frontoffice_url to compute orig parameter (#12381)