Development #13912
faire de lingo.check_request_signature une fonction dans utils
Début:
09 novembre 2016
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
lingo.check_request_signature(request) permet de vérifier si une signature reçue semble ok, ça sera utile dans d'autres API de combo, telle que celle des notifications (#13812)
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 7 ans
- Fichier 0001-move-check_request_signature-from-lingo-to-utils-139.patch 0001-move-check_request_signature-from-lingo-to-utils-139.patch ajouté
- Fichier 0002-tests-do-not-sign-an-already-signed-url.patch 0002-tests-do-not-sign-an-already-signed-url.patch ajouté
- Fichier 0003-utils.check_query-handle-missing-args-in-query.patch 0003-utils.check_query-handle-missing-args-in-query.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
le patch, accompagné de deux corrections associées au passage
- les tests qui testent pas très bien, parce que double sign_url(sign_url(url))
- et un if pour éviter de check_query crash quand la query ne contient pas les éléments nécessaires à une signature (découvert dans les tests de notifications)
Mis à jour par Thomas Noël il y a plus de 7 ans
- Lié à Development #13812: Système de notification ajouté
Mis à jour par Frédéric Péters il y a plus de 7 ans
Pour le 0003, détail mais je trouve plus facile à lire if not ('signature' in parsed and 'algo' in parsed ...). Avec ou sans ça, ack.
Mis à jour par Thomas Noël il y a plus de 7 ans
- Statut changé de En cours à Résolu (à déployer)
J'ai contacté George Boole et il était d'accord avec toi.
commit f2c27973ef41c0263fdd9c7f503d2ebd0d52009a
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Nov 9 01:19:10 2016 +0100
utils.check_query: handle missing args in query
commit db950321b5dce9f201fbcfc1ea0456c07ffd6008
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Nov 9 01:16:54 2016 +0100
tests: do not sign an already signed url
commit 2205f087168fa1ecc9f60a3849001667021b288e
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Nov 9 01:14:50 2016 +0100
move check_request_signature from lingo to utils (#13912)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
move check_request_signature from lingo to utils (#13912)