Project

General

Profile

Bug #20185

Erreur généré sur changement de mot de passe via l'admin

Added by Pierre Cros about 2 years ago. Updated almost 2 years ago.

Status:
Nouveau
Priority:
Normal
Category:
-
Target version:
Start date:
21 Nov 2017
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

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

History

#1 Updated by Benjamin Dauvergne about 2 years ago

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 Updated by Benjamin Dauvergne about 2 years ago

  • Target version set to future

#3 Updated by Frédéric Péters almost 2 years ago

{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 Updated by Benjamin Dauvergne almost 2 years ago

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 Updated by Frédéric Péters almost 2 years ago

Tout à fait pour; le fait qu'ils ne soient pas directement/pour de vrai supprimés ne devrait pas transparaitre.

Also available in: Atom PDF