Development #66266
authenticators, garder FranceConnect en premier
0%
Description
Sur une nouvelle instance, à l'ajout de FranceConnect, il n'apparaîtra pas forcément en premier car l'ordre est simplement hérité du modèle de base
order = models.IntegerField(_('Order'), default=0)
Avant, l'ordre était systématiquement mis à -1.
Est-ce que je fais un patch pour mettre -1 par défaut lors de l'ajout de FranceConnect ?
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 2 ans
Je dirais que non, je préfère l'uniformité dans le comportement, on ajoute un moyen d'authent, il se met derrière les autres, ils peuvent facilement être réordonnés après.
Mis à jour par Valentin Deniaud il y a presque 2 ans
J'ai de vagues souvenirs que dans le cas de FranceConnect la décision de l'avoir en premier avait une teneur politique, il faudrait qu'un CPF passe par là pour dire.
Précision technique,
on ajoute un moyen d'authent, il se met derrière les autres
actuellement ça ne fonctionne pas comme ça, c'est « on ajoute un moyen d'authent, il se met à l'index 0, c'est à dire qu'on a aucune idée d'où il est (il peut y avoir du -1, d'autres à 0...). Et on ne peut pas faire autrement parce que tout n'est pas encore en base. Ce sera l'objet de #65479.
Mis à jour par Frédéric Péters il y a presque 2 ans
on ajoute un moyen d'authent, il se met derrière les autres
actuellement ça ne fonctionne pas comme ça
Ok mais c'est le comportement qui aura, à terme, du sens. Pas "aucune idée d'où il est". (alternativement on peut aussi dire que l'ajout met en premier plutôt qu'en dernier, mais ça me parait moins naturel).
J'ai de vagues souvenirs que dans le cas de FranceConnect la décision de l'avoir en premier avait une teneur politique, il faudrait qu'un CPF passe par là pour dire.
Oui c'est un choix pas technique, et avoir un comportement cohérent qui ajoute à la fin obligera un glisser-déposer pour mettre FC en premier.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Dupliqué par Development #67513: sur l'ajout de l'authentification FC avoir par défaut un ordre pour qu'elle soit la première ajouté
Mis à jour par Valentin Deniaud il y a plus d'un an
Finalement,
Frédéric Péters a écrit :
C'est une exigence d'intégration et même si ça n'est pas compliqué à changer ça demande d'y penser à chaque fois, ce qui n'est sans doute pas tenable.
Donc le plan c'est toujours de glisser/déposer pour configurer l'ordre (#65479). Ce processus pose toujours des ordres >= 0, donc à la création forcer l'ordre de FC à -1 devrait suffire à toujours l'avoir premier.
Mis à jour par Frédéric Péters il y a plus d'un an
Super mon incohérence… Bref oui j'apparais résigné, que FranceConnect au contraire des autres, apparaisse en premier, partons là-dessus.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_fc-set-authenticator-to-be-first-by-default-on-.patch 0001-auth_fc-set-authenticator-to-be-first-by-default-on-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Voilà, mini patch basé sur #65479 qui introduit le dragndrop pour l'ordre et a un test test_authenticators_order qui montre qu'on peut toujours mettre FC ailleurs si on veut.
Mis à jour par Serghei Mihai 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 0bdf295d7d08df889a86f22155b07717b9faeabd Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Jul 20 15:48:22 2022 +0200 auth_fc: set authenticator to be first by default on creation (#66266)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
auth_fc: set authenticator to be first by default on creation (#66266)