Développement #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é.
Files
Related issues
Associated revisions
History
Updated by Paul Marillonnet over 2 years ago
- Related to Développement #65483: configuration des méthodes d'authentification, SAML added
Updated by Frédéric Péters over 2 years ago
attention authentic2-auth-fedict,
$ git grep authenticator ... src/authentic2_auth_fedict/authenticators.py:from authentic2.authenticators import BaseAuthenticator
Updated by Paul Marillonnet over 2 years 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 ?
Updated by Paul Marillonnet over 2 years 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 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.)
Updated by Paul Marillonnet over 2 years ago
- Related to Développement #66876: migrer FedictAuthenticator et sa config en base added
Updated by Valentin Deniaud over 2 years ago
- File 0001-misc-remove-dead-authenticators-code-66853.patch 0001-misc-remove-dead-authenticators-code-66853.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
À passer au (début du) prochain cycle, mais ça fait toujours plaisir.
Updated by Benjamin Dauvergne over 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud over 2 years 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)Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
Updated by Paul Marillonnet over 2 years 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.
Updated by Frédéric Péters over 2 years ago
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)