Project

General

Profile

Development #66853

virer authentic2.authenticators

Added by Paul Marillonnet (retour → 05 sept) about 2 months ago. Updated about 13 hours ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
01 July 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Avec #65483, tout est passé dans authentic2.apps.authenticators.models.
On pourra virer authentic2.authenticators, qui sera alors inutilisé.


Files


Related issues

Related to Authentic 2 - Development #65483: configuration des méthodes d'authentification, SAMLSolution déployée19 May 2022

Actions
Related to Plugin Carte eID (fedict/bosa) - Development #66876: migrer FedictAuthenticator et sa config en baseSolution déployée01 July 2022

Actions

Associated revisions

Revision bf400ae5 (diff)
Added by Valentin Deniaud 1 day ago

misc: remove dead authenticators code (#66853)

History

#1

Updated by Paul Marillonnet (retour → 05 sept) about 2 months ago

  • Related to Development #65483: configuration des méthodes d'authentification, SAML added
#2

Updated by Frédéric Péters about 2 months ago

attention authentic2-auth-fedict,

$ git grep authenticator
...
src/authentic2_auth_fedict/authenticators.py:from authentic2.authenticators import BaseAuthenticator
#3

Updated by Paul Marillonnet (retour → 05 sept) about 2 months ago

Frédéric Péters a écrit :

attention authentic2-auth-fedict,

Bien vu, décidément ce sera mon running gag GI.
J’ai l’impression que côté fedict/eid/bosa c’est quelque chose d’assez figé au niveau de la configuration SAML, qu’on a pas du tout intérêt à porter ça dans le manage, qu’on pourrait se contenter de réabsorber dans la classe fille FedictAuthenticator la classe parente BaseAuthenticator issue d’authentic, en supprimant l’héritage, et enfin faire ce ticket ci à proprement parler. Je me plante ?

#4

Updated by Paul Marillonnet (retour → 05 sept) about 2 months ago

Paul Marillonnet a écrit :

Bien vu, décidément ce sera mon running gag GI.
J’ai l’impression que côté fedict/eid/bosa c’est quelque chose d’assez figé au niveau de la configuration SAML, qu’on a pas du tout intérêt à porter ça dans le manage, qu’on pourrait se contenter de réabsorber dans la classe fille FedictAuthenticator la classe parente BaseAuthenticator issue d’authentic, en supprimant l’héritage, et enfin faire ce ticket ci à proprement parler. Je me plante ?

(À noter qu’on peut aussi créer un modèle qui sera très proche de celui fait pour SAML, écrire la migration, virer les settings, sans pour autant exposer un formulaire dans le BO.)

#5

Updated by Paul Marillonnet (retour → 05 sept) about 2 months ago

#6

Updated by Valentin Deniaud about 1 month ago

  • Assignee set to Valentin Deniaud
#7

Updated by Valentin Deniaud 27 days ago

À passer au (début du) prochain cycle, mais ça fait toujours plaisir.

#8

Updated by Benjamin Dauvergne → en congés jusqu'au 29/08 23 days ago

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

Updated by Valentin Deniaud 1 day ago

  • Status changed from Solution validée to Résolu (à déployer)
commit bf400ae52b6d0c5c267a45da1a73173986fc7642
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Jul 19 18:15:04 2022 +0200

    misc: remove dead authenticators code (#66853)
#10

Updated by Transition automatique 1 day ago

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

Updated by Frédéric Péters 1 day ago

Ça a pété l'authent 3M, #68199. (je m'en occupe)

#12

Updated by Paul Marillonnet (retour → 05 sept) about 13 hours ago

J’avais complètement loupé qu’il y aurait des choses à regarder côté thème(s) aussi. Merci d’avoir géré ça.

#13

Updated by Frédéric Péters about 13 hours ago

C'est juste une très ancienne intégration graphique et on voit ça en recette, pas de problème.

Also available in: Atom PDF