Développement #33992
permettre de passer request à authenticate()
Start date:
14 June 2019
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
Description
Cela ne marche toujours pas avec le backend de modèle (à cause du **kwargs).
Files
Associated revisions
add request as first argument to all backends (#33992)
History
Updated by Benjamin Dauvergne over 5 years ago
- File 0002-add-request-as-first-argument-to-all-backends-33992.patch 0002-add-request-as-first-argument-to-all-backends-33992.patch added
- File 0001-remove-auth2_ssl-33992.patch 0001-remove-auth2_ssl-33992.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
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
Updated by Valentin Deniaud over 5 years ago
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é)
Updated by Paul Marillonnet over 5 years ago
- Status changed from Solution proposée to Solution validée
Ok pour moi.
Updated by Benjamin Dauvergne over 5 years ago
- Status changed from Solution validée to 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)Updated by Frédéric Péters over 5 years ago
- Status changed from Résolu (à déployer) to Solution déployée
remove auth2_ssl (#33992)