Development #33992
permettre de passer request à authenticate()
Début:
14 juin 2019
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Cela ne marche toujours pas avec le backend de modèle (à cause du **kwargs).
Fichiers
Révisions associées
add request as first argument to all backends (#33992)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Fichier 0002-add-request-as-first-argument-to-all-backends-33992.patch 0002-add-request-as-first-argument-to-all-backends-33992.patch ajouté
- Fichier 0001-remove-auth2_ssl-33992.patch 0001-remove-auth2_ssl-33992.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Histoire d'avancer dans une compatibilité première vers Django 1.11 et parce que je vais avoir besoin de passer request à authenticate coté FranceConnect.
Au passage je vire auth2_ssl parce que :- aucun test
- aucun utilisateur
- pas assez de code pour ne pas être réécrit en 1 ou 2 jours proprement
- ça me crève le coeur de corrige une méthode authenticate() qui certainement ne fonctionne pas
Mis à jour par Valentin Deniaud il y a presque 5 ans
Benjamin Dauvergne a écrit :
Au passage je vire auth2_ssl
Une relique ici :
--- a/src/authentic2/idp/saml/saml2_endpoints.py
+++ b/src/authentic2/idp/saml/saml2_endpoints.py
@@ -364,10 +364,6 @@ def build_assertion(request, login, nid_format='transient'):
elif how == 'password-on-https':
authn_context = \
lasso.SAML2_AUTHN_CONTEXT_PASSWORD_PROTECTED_TRANSPORT
- elif how == 'ssl':
- authn_context = lasso.SAML2_AUTHN_CONTEXT_X509
- elif how.startswith('oath-totp'):
- authn_context = lasso.SAML2_AUTHN_CONTEXT_TIME_SYNC_TOKEN
else:
raise NotImplementedError('Unknown authentication method %s', how)
except ObjectDoesNotExist:
(oui, le bout avec totp avait été oublié lors d'une suppression analogue à celle-ci, pour ça que j'ai tilté)
Mis à jour par Paul Marillonnet il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Ok pour moi.
Mis à jour par Benjamin Dauvergne il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 27702e72c7c452112b3f885e00e223427fb3e77e
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Thu Jun 13 12:34:18 2019 +0200
add request as first argument to all backends (#33992)
commit 8bb83c01e79edcf91a0fd3e42bd70f7108ac77a6
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date: Fri Jun 14 14:38:37 2019 +0200
remove auth2_ssl (#33992)Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
remove auth2_ssl (#33992)