Produits Entr'ouvert » w.c.s.

Le cronjob pour nettoyer les nonces s'était perdu dans l'autre patch.

Voilà, avec des bouts du #10922, suffisamment pour que celui-ci s'applique. J'ai viré le sleep dans les tests qui faisait perdre 6s (à la place j'ai ajouté la possibilité de passer l'heure courante en paramètre à clean_nonces()). Je n'ai relancé que tests/test_api.py mais tout passe.

Rebasé et correction d'un bug suite au passage à pytest > 3 (pytest.raises() nettoie sys.exc_*, il faut utiliser la valeur renvoyer par le context manager à la place).

diff --git a/wcs/api_utils.py b/wcs/api_utils.py
index eaef3fa..d5b61c8 100644
--- a/wcs/api_utils.py
+++ b/wcs/api_utils.py
@@ -33,7 +33,7 @@ DEFAULT_DURATION = 30

 def is_url_signed(utcnow=None, duration=DEFAULT_DURATION):
-    if getattr(get_request(), 'signed', False):
+    if get_request().signed:
         return True
     query_string = get_request().get_query()
     if not query_string:
diff --git a/wcs/qommon/http_request.py b/wcs/qommon/http_request.py
index 2cc4591..9efda02 100644
--- a/wcs/qommon/http_request.py
+++ b/wcs/qommon/http_request.py
@@ -24,6 +24,8 @@ from quixote.errors import RequestError
 from http_response import HTTPResponse

 class HTTPRequest(quixote.http_request.HTTPRequest):
+    signed = False
+
     def __init__(self, *args, **kwargs):
         quixote.http_request.HTTPRequest.__init__(self, *args, **kwargs)
         self.response = HTTPResponse()

Ack.