Projet

Général

Profil

Bug #20185

/manage/ : ne pas lister les utilisateurs en cours de suppression

Ajouté par Pierre Cros il y a plus de 6 ans. Mis à jour il y a environ 4 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision 77a6c7b0 (diff)
Ajouté par Frédéric Péters il y a environ 4 ans

manager: don't list users being removed (#20185)

Historique

#1

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.

#2

Mis à jour par Benjamin Dauvergne il y a plus de 6 ans

  • Version cible mis à future
#3

Mis à jour par Frédéric Péters il y a plus de 6 ans

{u'#7348896': (501, '<#7348896>: malformed address: #7348896> may not follow <pcros │')}
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)
#4

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 ?

#5

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.

#6

Mis à jour par Frédéric Péters il y a environ 4 ans

Je passais par hasard et j'adapte l'intitulé aux derniers commentaires. Et tant qu'à faire, le patch.

#7

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.

#8

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

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

Formats disponibles : Atom PDF