Development #68383
authentificateur saml, supprimer le renommage des attributs
0%
Description
C'est bizarre voire incompréhensible d'avoir ça dans l'interface, et ça ne sert plus à rien maintenant qu'on a viré les conditions pour l'ajout de rôle.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_saml-remove-rename-attribute-action-68383.patch 0001-auth_saml-remove-rename-attribute-action-68383.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus d'un an
Dans les tests j'aurais laissé l'attribut SAML « 'http://nice/attribute/givenName': ['John'], » plutôt que le renommer d'office en « first_name »... juste histoire de voir que les tests marchent bien avec un attribut bizarre. (et pour l'avenir, que les tests tournent avec ce genre d'attribut).
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_saml-remove-rename-attribute-action-68383.patch 0001-auth_saml-remove-rename-attribute-action-68383.patch ajouté
Thomas Noël a écrit :
Dans les tests j'aurais laissé l'attribut SAML « 'http://nice/attribute/givenName': ['John']
Revoici
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-auth_saml-remove-rename-attribute-action-68383.patch 0001-auth_saml-remove-rename-attribute-action-68383.patch ajouté
J'avais zappé de virer la présentation des champs du template, aussi rebasé parce que tout ça a pas mal gébou.
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
C'est jeudi soir et il y a une migration, + déjà beaucoup de modifs dans authentic, j'enverrai au prochain cycle.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit a2568a45b5f389f958909151363df9e24c54fc15 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Aug 24 17:06:10 2022 +0200 auth_saml: remove rename attribute action (#68383)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Je souhaiterai une revert de ce ticket, si des conditions sont ré-introduites, et elles le seront, on ne pourra pas faire sans.
PS: et des templates aussi, enfin c'est juste chiant de supprimer des fonctionnalités gratuitement.
Mis à jour par Valentin Deniaud il y a plus d'un an
Benjamin Dauvergne a écrit :
Je souhaiterai une revert de ce ticket
C'est un revert compliqué à écrire, le code a déjà pas mal bougé.
si des conditions sont ré-introduites, et elles le seront, on ne pourra pas faire sans.
Pourquoi ne pas voir ça au moment où les conditions seront réintroduites ?
PS: et des templates aussi, enfin c'est juste chiant de supprimer des fonctionnalités gratuitement.
Pour moi le renommage se justifiait par les conditions, les conditions c'était beaucoup de code inutilisé, tout ça n'a pas été enlevé gratuitement mais bien pour des raisons de maintenance/simplification à la fois du code et de l'interface.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Valentin Deniaud a écrit :
C'est un revert compliqué à écrire, le code a déjà pas mal bougé.
Je vois, mais maintenant que c'est juste 10x plus difficile de l'ajouter parce qu'il faut faire un modèle, une vue d'édition, etc.. c'est sûr.
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
auth_saml: remove rename attribute action (#68383)