Development #32567
authn oidc : inclure un code de statut dans les réponses httmock
Début:
24 avril 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Parce que, si mes souvenirs sont bons, on fait dans le code un raise_for_status de la réponse, qui vérifie notamment que la valeur du code HTTP de retour est bien strictement inférieure à 400 (valeur à laquelle commencent les codes d'erreur).
Le champ status_code de l'objet de réponse vaut None.
En python2, pas de souci, le raise_for_status ne va rien déceler d'anormal :
In [1]: assert None < 400 In [2]:
Par contre, en python3, c'est un peu plus rigoureux :
In [1]: assert None < 400 --------------------------------------------------------------------------- TypeError Traceback (most recent call last) <ipython-input-1-7401c835355d> in <module>() ----> 1 assert None < 400 TypeError: '<' not supported between instances of 'NoneType' and 'int' In [2]:
Fichiers
Historique
Mis à jour par Paul Marillonnet il y a presque 5 ans
- Fichier 0001-oidc-authn-include-a-status-code-in-httmock-response.patch 0001-oidc-authn-include-a-status-code-in-httmock-response.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution proposée à En cours
Tests cassés.
Mis à jour par Paul Marillonnet il y a presque 5 ans
- Fichier 0001-oidc-authn-include-a-status-code-in-httmock-response.patch 0001-oidc-authn-include-a-status-code-in-httmock-response.patch ajouté
- Statut changé de En cours à Solution proposée
Erreur de timing de ma part, ce patch doit s'appliquer après celui de #32568-4, et donc avec une modification dans les tests de la chaîne recherchée lors d'une erreur client à l'obtention d'un jeton d'accès.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Paul Marillonnet
À rebaser sans #32568.
Mis à jour par Paul Marillonnet il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Ok.
commit 0e923ba857c697c8a2c26868142857b390696ac0
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date: Wed Apr 10 17:11:28 2019 +0200
oidc authn: include a status code in httmock responses