Project

General

Profile

Development #33992

permettre de passer request à authenticate()

Added by Benjamin Dauvergne 12 months ago. Updated 11 months ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
14 Jun 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

Cela ne marche toujours pas avec le backend de modèle (à cause du **kwargs).

0002-add-request-as-first-argument-to-all-backends-33992.patch View (15.8 KB) Benjamin Dauvergne, 14 Jun 2019 03:24 PM

0001-remove-auth2_ssl-33992.patch View (47.3 KB) Benjamin Dauvergne, 14 Jun 2019 03:24 PM

Associated revisions

Revision 8bb83c01 (diff)
Added by Benjamin Dauvergne 11 months ago

remove auth2_ssl (#33992)

Revision 27702e72 (diff)
Added by Benjamin Dauvergne 11 months ago

add request as first argument to all backends (#33992)

History

#1 Updated by Benjamin Dauvergne 12 months ago

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

#2 Updated by Valentin Deniaud 12 months 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é)

#3 Updated by Paul Marillonnet 11 months ago

  • Status changed from Solution proposée to Solution validée

Ok pour moi.

#4 Updated by Benjamin Dauvergne 11 months 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)

#5 Updated by Frédéric Péters 11 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF