Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adresse email avec des caractères spéciaux provoque des 500 avec gravatar #1640

Closed
gustavi opened this issue Oct 28, 2014 · 8 comments
Closed
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Milestone

Comments

@gustavi
Copy link
Contributor

gustavi commented Oct 28, 2014

On a un membre qui a une adresse email avec un caractère spécial dedans et ça lève une exception quand on appelle cette ligne : https://github.com/zestedesavoir/zds-site/blob/dev/zds/member/models.py#L123.

L'email ressemble à : <foo>@\xfbgmail.com

Le gros souci c'est que ça affecte toutes les pages où il y a l'avatar de ce membre.

Est-ce qu'il est normal qu'il puisse avoir une adresse email comme celle-ci ?
Si ce n'est pas normal, comment a-t-il fait ? (en modifiant son adresse mail ?)

Il faut également régler ce problème d'UTF8 sur l'appel get_avatar_url (au niveau de gravatar).

Les logs : http://paste.awesom.eu/0hjp

@Eskimon
Copy link
Contributor

Eskimon commented Oct 28, 2014

Est-ce qu'il est normal qu'il puisse avoir une adresse email comme celle-ci ?

A priori dorénavant oui : http://fr.wikipedia.org/wiki/Adresse_%C3%A9lectronique

En gros, ne suffirait-il pas de passer simplement en unicode la chaîne (rajout du 'u').

@Eskimon Eskimon added S-BUG Corrige un problème C-Back Concerne le back-end Django labels Oct 28, 2014
@Eskimon Eskimon added this to the "Futur proche" (v1.x) milestone Oct 28, 2014
@SpaceFox
Copy link
Contributor

Je crois qu'il va falloir faire un patch "unicode", qui colle des "u"
devant TOUTES les chaînes de caractères du code.

Le 28 octobre 2014 11:44, Eskimon notifications@github.com a écrit :

Est-ce qu'il est normal qu'il puisse avoir une adresse email comme
celle-ci ?

A priori dorénavant oui :
http://fr.wikipedia.org/wiki/Adresse_%C3%A9lectronique

En gros, ne suffirait-il pas de passer simplement en unicode la chaîne
(rajout du 'u').


Reply to this email directly or view it on GitHub
#1640 (comment)
.

@gustavi
Copy link
Contributor Author

gustavi commented Oct 28, 2014

Ou alors on passe à python3 et plus de problème !

@SpaceFox
Copy link
Contributor

Si tu corriges les dépendances qui ne supportent pas Python 3, ça peut se
faire !

2014-10-28 11:52 GMT+01:00 Laville Augustin notifications@github.com:

Ou alors on passe à python3 et plus de problème !


Reply to this email directly or view it on GitHub
#1640 (comment)
.

@GerardPaligot
Copy link
Member

Je relance cette issue parce qu'elle n'a toujours pas été corrigée (malgré une tentative de PR) et qu'elle est toujours en prod.

@Eskimon
Copy link
Contributor

Eskimon commented Jan 14, 2015

Le souci c'est que si on la résout c'est du cote de git que ca merde... Donc il y a deux solutions:

  • Passer a Python 3 :)
  • Interdire les caractères alakon dans les emails

@Xalfen
Copy link

Xalfen commented Jan 14, 2015

Interdire les caractères spéciaux dans les adresses emails, c'est aussi explicitement ne pas appliquer les RFC correspondantes. Perso, j'utilise les "+" dans mes emails pour me retrouver dans les différents sites où je suis inscrit (et aussi savoir d'où vient un mail si ton adresse a été leakée par exemple). Je trouverais ça un bête

@Eskimon Eskimon modified the milestones: Version 1.7, "Futur proche" (v1.x) Mar 19, 2015
@Eskimon
Copy link
Contributor

Eskimon commented Mar 19, 2015

Enfin corrigé :) Merci à tous !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

6 participants