Development #36509
rendre PassiveAuthenticationMiddleware compatible avec Django 2.2
0%
Description
Comme ça ne fait pas partie des tests ça a été sauté (donc aussi ajouter des tests).
Fichiers
Demandes liées
Révisions associées
use MiddlewareMixin on middleware (#36509)
Remove OPENED_SESSION_COOKIE_DOMAIN which has no use.
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-factorize-compatibility-layer-36509.patch 0001-factorize-compatibility-layer-36509.patch ajouté
- Fichier 0002-use-MiddlewareMixin-on-middleware-36509.patch 0002-use-MiddlewareMixin-on-middleware-36509.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 4 ans
On pourrait juste virer la compat < 1.11, c'est ce qu'on disait lundi dernier.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-factorize-compatibility-layer-36509.patch 0001-factorize-compatibility-layer-36509.patch ajouté
- Fichier 0002-use-MiddlewareMixin-on-middleware-36509.patch 0002-use-MiddlewareMixin-on-middleware-36509.patch ajouté
Manque un fichier.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Frédéric Péters a écrit :
On pourrait juste virer la compat < 1.11, c'est ce qu'on disait lundi dernier.
La compat 1.8 ne coûte pas grand chose (10 lignes) et le code de la lib elle même n'est pas bien long et ne risque pas d'énormément évoluer, comme c'est quasiment la seule implémentation SAML/Lasso/Django1 je la laisserai comme cela pour l'instant, sait-on jamais.
1 j'ai trouvé ça https://github.com/IronCountySchoolDistrict/django-python3-saml mais c'est loin de fournir les mêmes services
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Fichier 0001-factorize-compatibility-layer-36509.patch 0001-factorize-compatibility-layer-36509.patch ajouté
- Fichier 0002-use-MiddlewareMixin-on-middleware-36509.patch 0002-use-MiddlewareMixin-on-middleware-36509.patch ajouté
WSGI est un peu chiant avec des chaînes unicode.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Bloque Development #36584: Support MIDDLEWARE nouveau style dans debian_config_common.py ajouté
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
J'ai l'impression qu'il manque 'attrs<=19.1.0' dans le tox.ini pour pouvoir jouer les tests en local.
Après, je n'arrive pas à passer les tests en python3 mais je crois que c'est pas lié à django-mellon.
ModuleNotFoundError: No module named 'lasso'
Sinon, testé le patch et je ne constate pas de régression.
Pour le code, la partie concernant OPENED_SESSION_COOKIE_DOMAIN qui ajoutait une restriction supplémentaire
# accept automatic common domain selection if domain has at least three components # and is not an IP address
me semble effectivement redondante avec
if app_settings.OPENED_SESSION_COOKIE_NAME in request.COOKIES:
Puisque les cookies sont par définition associés à (ou livré suivant) un domaine.
Donc je valide si ça peut débloquer, mais bon je rentre tout juste dans ce code.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Nicolas Roche a écrit :
J'ai l'impression qu'il manque 'attrs<=19.1.0' dans le tox.ini pour pouvoir jouer les tests en local.
C'est quoi le souci avec attrs ? J'ai du rater quelque chose.
Après, je n'arrive pas à passer les tests en python3 mais je crois que c'est pas lié à django-mellon.
[...]
Sur jenkins ça passe.
Sinon, testé le patch et je ne constate pas de régression.
Pour le code, la partie concernant OPENED_SESSION_COOKIE_DOMAIN qui ajoutait une restriction supplémentaire
[...]
me semble effectivement redondante avec
[...]
Puisque les cookies sont par définition associés à (ou livré suivant) un domaine.
Tout à fait, je ne sais pas ce qui m'est passé par la tête ici.
Donc je valide si ça peut débloquer, mais bon je rentre tout juste dans ce code.
Ok.
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 7095b1368b4c3c58f6669f4f6260c8c9ed15f056 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Sun Sep 29 11:20:44 2019 +0200 use MiddlewareMixin on middleware (#36509) Remove OPENED_SESSION_COOKIE_DOMAIN which has no use. commit 83abc786058a3cf2258980aaa822d12f2eaca156 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Sun Sep 29 11:20:02 2019 +0200 factorize compatibility layer (#36509)
Mis à jour par Frédéric Péters il y a plus de 4 ans
C'est quoi le souci avec attrs ? J'ai du rater quelque chose.
Désormais incompatible avec les projets où pytest avait été limité à une certaine version à cause d'un bug passé dans un des plugins pytest, etc.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
factorize compatibility layer (#36509)