Projet

Général

Profil

Development #41262

signature.py : ne pas obliger les nonces si y'a pas de vérificateur, et autres corrections

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

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
02 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Ca vient surtout de choses vues lors de #41245


Fichiers

Historique

#1

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

On note au passage qu'un test était juste faux, il aurait dû planter :

   assert signature.check_url(signature.sign_url(URL, KEY, nonce='xxx'), KEY,
           known_nonce=lambda nonce: nonce == 'xxx')

c'est corrigé ici.

#2

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

Ça build pas.

#3

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

En fait ça ne builde pas sur un problème Authentic (#41284 ?)

ERROR:   py2-authentic: commands failed
  py2-hobo: commands succeeded
  py2-multipublik: commands succeeded
  py2-multitenant: commands succeeded
  py2-multitenant-oldstylemiddleware: commands succeeded
  py2-passerelle: commands succeeded
  py2-schemas: commands succeeded
ERROR:   py3-coverage-authentic: commands failed
  py3-coverage-hobo: commands succeeded
  py3-coverage-multipublik: commands succeeded
  py3-coverage-multitenant: commands succeeded
  py3-coverage-multitenant-oldstylemiddleware: commands succeeded
  py3-coverage-passerelle: commands succeeded
  py3-coverage-schemas: commands succeeded
#4

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

  • Statut changé de Solution proposée à Solution validée

Ouaip, je valide donc.

#5

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

  • Statut changé de Solution validée à Résolu (à déployer)

Merci.

commit df903c40b3dd3fe1ccd9eefa302e0fc8613eb4a1
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Thu Apr 2 13:50:36 2020 +0200

    signature: do not require nonce if not verified, and other fixes (#)

#6

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

(et zut, numéro de ticket oublié dans le commit... tant pis)

#7

Mis à jour par Frédéric Péters il y a environ 2 ans

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

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

Automatic expiration

Formats disponibles : Atom PDF