Bug #75286
clean-unused-accounts : parmi les usagers inactifs, la sélection de ceux à supprimer ne se fait pas sur le keepalive
0%
Description
Dans le code de la commande de nettoyage des comptes inactifs :
inactive_users = ou_users.filter(
(
Q(last_login__lte=self.now - alert_delay)
| (Q(last_login__isnull=True) & Q(date_joined__lte=self.now - alert_delay))
)
& (Q(keepalive__isnull=True) | Q(keepalive__lte=self.now - alert_delay))
)
mais plus loin dans le code :
inactive_users_to_delete = inactive_users.filter(
(
Q(last_login__lte=self.now - deletion_delay)
| Q(last_login__isnull=True) & Q(date_joined__lte=self.now - deletion_delay)
)
# […]
)
-> cette contrainte sur le keepalive a été perdue en cours de route, ce qui fait qu’on peut potentiellement supprimer à tort des usagers inactifs depuis suffisamment longtemps pour avoir reçu une alerte, mais pas assez longtemps pour mériter suppression.
Demandes liées
Révisions associées
Historique
Mis à jour par Paul Marillonnet il y a environ un an
- Statut changé de Nouveau à En cours
- Assigné à mis à Paul Marillonnet
- Priorité changé de Normal à Haut
C’est un peu pénible en fait, il faudrait corriger rapidement : il va y avoir des comptes supprimés à l’exécution du cron directement après celle ayant provoqué l’envoi de l’alerte. C’est quand même un bon gros bug.
Mis à jour par Paul Marillonnet il y a environ un an
- Lié à Development #67901: web-service 'keepalive' pour décaler l'expiration d'un compte ajouté
Mis à jour par Paul Marillonnet il y a environ un an
- Priorité changé de Haut à Normal
Bon, j’ai vu des fantômes je crois. Il me semble qu’il y a quand même des cas limites où il faudrait revoir ça, mais je regarderai à tête reposée. J’ai poussé une branche pour mémoire.
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de En cours à Solution proposée
Paul Marillonnet (pmarillonnet) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/19
- Titre : unused-accounts: use keepalive when identifying users to delete (#75286)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/19/files
Mis à jour par Paul Marillonnet il y a environ un an
Paul Marillonnet a écrit :
Bon, j’ai vu des fantômes je crois. Il me semble qu’il y a quand même des cas limites où il faudrait revoir ça, mais je regarderai à tête reposée. J’ai poussé une branche pour mémoire.
Et en fait c’est plus clair maintenant, je propose la PR avec l’explication qui va avec, sur gitea.
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution proposée à Solution validée
Benjamin Dauvergne (bdauvergne) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a environ un an
- Statut changé de Solution validée à Résolu (à déployer)
Paul Marillonnet (pmarillonnet) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/19
- Titre : unused-accounts: use keepalive when identifying users to delete (#75286)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/19/files
Mis à jour par Transition automatique il y a environ un an
- Statut changé de Résolu (à déployer) à Solution déployée
unused-accounts: use keepalive when identifying users to delete (#75286)