Projet

Général

Profil

Development #41930

utiliser un attribut pour marquer les comptes à supprimer

Ajouté par Benjamin Dauvergne il y a presque 4 ans. Mis à jour il y a presque 4 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
21 avril 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

On a pas besoin d'un modèle.


Fichiers


Demandes liées

Lié à Authentic 2 - Bug #40766: demande de suppression de compte depuis une session ouverte alors que le compte a entretemps été suppriméNouveau16 mars 2020

Actions
Bloque Authentic 2 - Development #41933: conserver une trace des comptes supprimésFermé21 avril 2020

Actions
Bloque Authentic 2 - Development #41948: À la suppression d'un compte, fermer toutes les sessions ouvertesNouveau22 avril 2020

Actions

Révisions associées

Révision a5d84366 (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

tests: add init files (#41930)

It prevents collisions between test files with the same basename.

Révision 9f0aa6ad (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

misc: replace DeletedUser model by attribute deleted on User (#41930)

Révision 5b07d440 (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

backends: prevent authentication by deleted user (#41930)

Révision 0f241a71 (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

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.

Révision 148642cc (diff)
Ajouté par Benjamin Dauvergne il y a presque 4 ans

misc: show deleted attribute in UserAdmin (#41930)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

#3

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
#4

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

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.

#5

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é
#6

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é
#7

Mis à jour par Benjamin Dauvergne il y a presque 4 ans

Rebasé.

#8

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.

#9

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.

#10

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/.

#11

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)
#12

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

Formats disponibles : Atom PDF