Produits Entr'ouvert » w.c.s.

Pourquoi supprimer le fallback sur l'abance de paramètre algo= ?

    MAX_DELTA = 30
    if abs(delta) > datetime.timedelta(seconds=MAX_DELTAT):

Modif visiblement pas testée.

Frédéric Péters a écrit :

Pourquoi supprimer le fallback sur l'abance de paramètre algo= ?

Ça n'est pas sain: si une personne utilise SHA1 et ne précise pas d'algo cela va systématiquement faire une erreur de signature alors que c'est l'algorithme qui n'est pas le bon et que la signature est bonne en fait (i.e. elle est bien calculée). J'espère juste que rien n'en dépend pour l'instant.

[...]

Modif visiblement pas testée.

Je tâtais le terrain.

Je tâtais le terrain.

Ok, sur une question particulière ? Parce que là Thomas fait le ticket, si c'est sur l'objet même du ticket qu'il y a hésitation, je ne vois pas le sens de faire un patch non testé.

(wcs)bdauvergne@fenouil:~/Code/wcs$ PYTHONPATH=$(pwd) py.test --run-postgresql tests/test_api.py
=============================================================================================== test session starts ================================================================================================
platform linux2 -- Python 2.7.3 -- py-1.4.24 -- pytest-2.6.2
collected 1 items 

tests/test_api.py .

============================================================================================= 1 passed in 0.59 seconds =============================================================================================
(wcs)bdauvergne@fenouil:~/Code/wcs$ fg 

Il y a un "import sys" volant dans les tests; c'est mieux d'avoir une série de tests qu'un méga long test; par exemple,

...

def test_missing_algo():
    output = visit_page('/user?format=json&orig=coucou&signature=xxx')
    content = ''.join(output.generate_body_chunks())
    result = json.loads(content)
    assert result['err_desc'] == 'missing/multiple algo field'

def test_invalid_algo():
    output = visit_page('/user?format=json&orig=coucou&signature=xxx&algo=coin')
    content = ''.join(output.generate_body_chunks())
    result = json.loads(content)
    assert result['err_desc'] == 'invalid algo'

...

Cool, ok.