Bug #20185
/manage/ : ne pas lister les utilisateurs en cours de suppression
0%
Description
J'ai joué avec ce compte :
https://connexion-lenord.test.entrouvert.org/manage/users/411/
Je l'ai supprimé en front, puis réactivé via l'admin :
- Erreur 1 : pas possible de se logguer pour l'usager ("mot de passe invalide")
- Erreur 2 : j'ai essayé de modifier le mot de passe dans l'admin https://connexion-lenord.test.entrouvert.org/manage/users/411/change-password/ "
Erreur de serveur. Nous sommes désolé mais nous constatons une erreur serveur."
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
Pierre Cros a écrit :
J'ai joué avec ce compte :
https://connexion-lenord.test.entrouvert.org/manage/users/411/Je l'ai supprimé en front, puis réactivé via l'admin :
- Erreur 1 : pas possible de se logguer pour l'usager ("mot de passe invalide")
It's a feature not a bug, le mot de passe est invalidé à la demande de suppression pour éviter que la personne puisse ré-activer le compte par ailleurs.
- Erreur 2 : j'ai essayé de modifier le mot de passe dans l'admin https://connexion-lenord.test.entrouvert.org/manage/users/411/change-password/ "
Erreur de serveur. Nous sommes désolé mais nous constatons une erreur serveur."
{u'pcros+test@entrouvert.com#7348896': (501, '<pcros+test@entrouvert.com#7348896>: malformed address: #7348896> may not follow <pcros │++test@entrouvert.com')}
Je ne sais pas trop comment le #7348896 s'est retrouvé là, si c'est un compte créé en BO faut savoir que les adresses de courriel ne reçoivent aucune validation à part la syntaxe de base (youpi il y a un @ au milieu).
Je garde le ticket pour cette histoire de validation et mail où au moins d'interception des erreurs SMTP à un moment.
Mis à jour par Frédéric Péters il y a plus de 6 ans
{u'pcros+test@entrouvert.com#7348896': (501, '<pcros+test@entrouvert.com#7348896>: malformed address: #7348896> may not follow <pcros │++test@entrouvert.com')}
Je ne sais pas trop comment le #7348896 s'est retrouvé là, si c'est un compte créé en BO faut savoir que les adresses de courriel ne reçoivent aucune validation à part la syntaxe de base (youpi il y a un @ au milieu).
C'est authentic qui fait cette modification à l'adresse email lors de la demande de suppression :
self.request.user.email += '#%d' % random.randint(1, 10000000)
Mis à jour par Benjamin Dauvergne il y a plus de 6 ans
D'ac bon c'est vraiment un cas limite, je propose de cacher les comptes en cours de suppression du BO dans ce cas, ça irait à tout le monde ?
Mis à jour par Frédéric Péters il y a plus de 6 ans
Tout à fait pour; le fait qu'ils ne soient pas directement/pour de vrai supprimés ne devrait pas transparaitre.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Fichier 0001-manager-don-t-list-users-being-removed-20185.patch 0001-manager-don-t-list-users-being-removed-20185.patch ajouté
- Sujet changé de Erreur généré sur changement de mot de passe via l'admin à /manage/ : ne pas lister les utilisateurs en cours de suppression
- Statut changé de Nouveau à Solution proposée
- Assigné à changé de Benjamin Dauvergne à Frédéric Péters
- Patch proposed changé de Non à Oui
Je passais par hasard et j'adapte l'intitulé aux derniers commentaires. Et tant qu'à faire, le patch.
Mis à jour par Benjamin Dauvergne il y a environ 4 ans
- Statut changé de Solution proposée à Solution validée
Il me semble que ça marche aussi avec deletion__isnull=True
.
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
Tout à fait, et modifié en ce sens,
commit 77a6c7b07c0b353d3f3dde2385a32307a367a744 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Feb 18 15:00:04 2020 +0100 manager: don't list users being removed (#20185)
Mis à jour par Frédéric Péters il y a environ 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
manager: don't list users being removed (#20185)