Development #52671
ldap: journaliser la désactivation des comptes LDAP orphelins
0%
Description
On ne sait pas actuellement pourquoi un compte a été désactivé.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a presque 3 ans
- Sujet changé de ldap: journaliser la désacivation des comptes LDAP orphelins à ldap: journaliser la désactivation des comptes LDAP orphelins
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Dupliqué par Bug #53981: la désactivation auto d'un utilisateur n'apparait pas dans le journal ajouté
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Dupliqué par Support #53236: journaliser la désactivation d'un compte faite ailleurs que dans backoffice ajouté
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Fichier 0001-journal_event_types-add-ldap-user-deactivation-52671.patch 0001-journal_event_types-add-ldap-user-deactivation-52671.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Pas d'objet request donc je suis parti sur l'utilisation directe de la classe de l'évènement, mais c'est peut-être mieux d'instancier un objet Journal et d'utiliser sa méthode record ?
Mis à jour par Benjamin Dauvergne il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Je n'ai pas de souci avec l'utilisation directe de record()
par contre je me demande si c'est bien utile de commencer une hiérarchie d'évènement ldap...
peut-être simplement user.deactivation
et mettre dans reason que ça vient du backend d'annuaire LDAP (genre si on a SCIM ou SQL ou que sais-je dans le futur ce ne sera pas un nouvel évènement), et peut-être ajouter un champ origine='ldaps://...'.
Je valide quand même mais je te laisse réfléchir au sujet.
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Fichier 0001-journal_event_types-add-ldap-user-deactivation-52671.patch 0001-journal_event_types-add-ldap-user-deactivation-52671.patch ajouté
- Statut changé de Solution validée à Solution proposée
Revoici sans évènement LDAP spécifique.
Mis à jour par Benjamin Dauvergne il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Je valide, je vais harmoniser la valeur de reason dans mon code de réactivation.
Mis à jour par Valentin Deniaud il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 8268e9e69c718716a44cb6d2888e032d05628ffd Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Mon May 17 14:23:35 2021 +0200 ldap: record user deactivation in journal (#52671)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ldap: record user deactivation in journal (#52671)