Development #41930
utiliser un attribut pour marquer les comptes à supprimer
0%
Description
On a pas besoin d'un modèle.
Fichiers
Demandes liées
Révisions associées
misc: replace DeletedUser model by attribute deleted on User (#41930)
backends: prevent authentication by deleted user (#41930)
misc: re-use logout view in deletion view (#41930)
As user is not accessible anymore through ModelBackend.get_user(), we
must call logout() directly, and not through a redirection.
misc: show deleted attribute in UserAdmin (#41930)
Historique
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Bloque Development #41933: conserver une trace des comptes supprimés ajouté
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Fichier 0001-backends-prevent-authentication-by-deleted-user.patch 0001-backends-prevent-authentication-by-deleted-user.patch ajouté
- Fichier 0004-misc-re-use-logout-view-in-deletion-view.patch 0004-misc-re-use-logout-view-in-deletion-view.patch ajouté
- Fichier 0003-misc-replace-DeletedUser-model-by-attribute-deleted-.patch 0003-misc-replace-DeletedUser-model-by-attribute-deleted-.patch ajouté
- Fichier 0002-backends-prevent-access-by-inactive-users.patch 0002-backends-prevent-access-by-inactive-users.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Sujet changé de utiliser une colonne pour marquer les comptes supprimés à utiliser un attribut pour marquer les comptes à supprimer
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Fichier 0001-backends-prevent-authentication-by-deleted-user.patch 0001-backends-prevent-authentication-by-deleted-user.patch ajouté
- Fichier 0002-misc-replace-DeletedUser-model-by-attribute-deleted-.patch 0002-misc-replace-DeletedUser-model-by-attribute-deleted-.patch ajouté
- Fichier 0003-misc-re-use-logout-view-in-deletion-view.patch 0003-misc-re-use-logout-view-in-deletion-view.patch ajouté
J'ai retiré la déconnection implicite des utilisateurs désactivés, je pensais corrigé au passage #40766 mais ça va se corriger tout seul par l'impossibilité de rester connecté si user.deleted est vrai.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Lié à Bug #40766: demande de suppression de compte depuis une session ouverte alors que le compte a entretemps été supprimé ajouté
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Bloque Development #41948: À la suppression d'un compte, fermer toutes les sessions ouvertes ajouté
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de Solution proposée à En cours
Encore quelques commits il y a plein de migrations manquantes dans a2.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de En cours à Solution proposée
Rebasé sur #42504 pour la correction aux migrations.
Mis à jour par Valentin Deniaud il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Dans 0002, tu fais un def noop()
alors qu'il existe RunPython.noop. Aussi il me semble que 0001 dépend de 0002, donc inverser les deux patches.
Je trouve que la migration aurait mérité un test, d'autant plus qu'on a un décorateur pour simplifier les tests de migrations, mais comme tu le sens, j'ai testé en local ça a marché. En faisant ça je me suis dis que le champ deleted a peut-être sa place dans l'interface /admin/.
Mis à jour par Benjamin Dauvergne il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 148642cceda67c4088823a8fac12ac53ae2efc77 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Thu May 7 23:46:42 2020 +0200 misc: show deleted attribute in UserAdmin (#41930) commit 0f241a712cdf30453e500908347906de6ab82e28 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Apr 21 22:36:00 2020 +0200 misc: re-use logout view in deletion view (#41930) As user is not accessible anymore through ModelBackend.get_user(), we must call logout() directly, and not through a redirection. commit 5b07d440bce00c78f6be125291fd0a9493ee85d9 Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Apr 21 22:26:50 2020 +0200 backends: prevent authentication by deleted user (#41930) commit 9f0aa6ad3c395a5aa9c0bce89d0df167903837fb Author: Benjamin Dauvergne <bdauvergne@entrouvert.com> Date: Tue Apr 21 22:32:54 2020 +0200 misc: replace DeletedUser model by attribute deleted on User (#41930)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: add init files (#41930)
It prevents collisions between test files with the same basename.