Development #66853
virer authentic2.authenticators
0%
Description
Avec #65483, tout est passé dans authentic2.apps.authenticators.models.
On pourra virer authentic2.authenticators, qui sera alors inutilisé.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Lié à Development #65483: configuration des méthodes d'authentification, SAML ajouté
Mis à jour par Frédéric Péters il y a presque 2 ans
attention authentic2-auth-fedict,
$ git grep authenticator ... src/authentic2_auth_fedict/authenticators.py:from authentic2.authenticators import BaseAuthenticator
Mis à jour par Paul Marillonnet il y a presque 2 ans
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 ?
Mis à jour par Paul Marillonnet il y a presque 2 ans
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 filleFedictAuthenticatorla classe parenteBaseAuthenticatorissue 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.)
Mis à jour par Paul Marillonnet il y a presque 2 ans
- Lié à Development #66876: migrer FedictAuthenticator et sa config en base ajouté
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-misc-remove-dead-authenticators-code-66853.patch 0001-misc-remove-dead-authenticators-code-66853.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
À passer au (début du) prochain cycle, mais ça fait toujours plaisir.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à 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)Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a plus d'un an
Ça a pété l'authent 3M, #68199. (je m'en occupe)
Mis à jour par Paul Marillonnet il y a plus d'un an
J’avais complètement loupé qu’il y aurait des choses à regarder côté thème(s) aussi. Merci d’avoir géré ça.
Mis à jour par Frédéric Péters il y a plus d'un an
C'est juste une très ancienne intégration graphique et on voit ça en recette, pas de problème.
misc: remove dead authenticators code (#66853)