Développement #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.
Files
Associated revisions
History
Updated by Valentin Deniaud over 2 years ago
- File 0001-auth_saml-remove-rename-attribute-action-68383.patch 0001-auth_saml-remove-rename-attribute-action-68383.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Thomas Noël (congés → 5 décembre) over 2 years ago
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).
Updated by Valentin Deniaud about 2 years ago
- File 0001-auth_saml-remove-rename-attribute-action-68383.patch 0001-auth_saml-remove-rename-attribute-action-68383.patch added
Thomas Noël a écrit :
Dans les tests j'aurais laissé l'attribut SAML « 'http://nice/attribute/givenName': ['John']
Revoici
Updated by Valentin Deniaud about 2 years ago
- File 0001-auth_saml-remove-rename-attribute-action-68383.patch 0001-auth_saml-remove-rename-attribute-action-68383.patch added
J'avais zappé de virer la présentation des champs du template, aussi rebasé parce que tout ça a pas mal gébou.
Updated by Serghei Mihai about 2 years ago
- Status changed from Solution proposée to Solution validée
Updated by Valentin Deniaud about 2 years ago
C'est jeudi soir et il y a une migration, + déjà beaucoup de modifs dans authentic, j'enverrai au prochain cycle.
Updated by Valentin Deniaud about 2 years ago
- Status changed from Solution validée to 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)Updated by Benjamin Dauvergne about 2 years ago
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.
Updated by Valentin Deniaud about 2 years ago
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.
Updated by Benjamin Dauvergne about 2 years ago
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.
Updated by Transition automatique about 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
auth_saml: remove rename attribute action (#68383)