email commençant par un espace est rejeté sans explication
#5218
Closed
opened 2 months ago by JLuc
·
14 comments
No Branch/Tag Specified
1.8
1.9.1
1.9.2
2.0
2.1
3.0
3.1
3.2
4.0
4.1
boutons-danger
debug_ecrire_fichier
dev-sortable
dev/autoloader
dev/instituer_ergo
dev_infos_image
fix/valider_url_distante
fix_modifier_login
issue_4101
issue_4678
issue_4705
issue_4717
issue_4946
issue_5016
issue_5056_composer_road
issue_5273
master
refactor_texte_safety
v1.8.3+b
v1.9.1+i
v1.9.2+f
v1.9.2+g
v1.9.2+h
v1.9.2+i
v1.9.2+j
v1.9.2+k
v1.9.2+m
v1.9.2+n
v1.9.2+o
v1.9.2+p
v2.0.0
v2.0.1
v2.0.10
v2.0.11
v2.0.12
v2.0.13
v2.0.14
v2.0.15
v2.0.16
v2.0.17
v2.0.18
v2.0.19
v2.0.2
v2.0.20
v2.0.21
v2.0.22
v2.0.23
v2.0.24
v2.0.25
v2.0.26
v2.0.3
v2.0.5
v2.0.6
v2.0.7
v2.0.8
v2.0.9
v2.1.0
v2.1.1
v2.1.10
v2.1.11
v2.1.12
v2.1.13
v2.1.14
v2.1.15
v2.1.16
v2.1.17
v2.1.18
v2.1.19
v2.1.2
v2.1.20
v2.1.21
v2.1.22
v2.1.23
v2.1.24
v2.1.25
v2.1.26
v2.1.27
v2.1.28
v2.1.29
v2.1.3
v2.1.30
v2.1.4
v2.1.5
v2.1.6
v2.1.7
v2.1.8
v2.1.9
v3.0.0
v3.0.0-alpha.1
v3.0.0-beta
v3.0.0-beta.2
v3.0.0-rc
v3.0.1
v3.0.10
v3.0.11
v3.0.12
v3.0.13
v3.0.14
v3.0.15
v3.0.16
v3.0.17
v3.0.18
v3.0.19
v3.0.2
v3.0.20
v3.0.21
v3.0.22
v3.0.23
v3.0.24
v3.0.25
v3.0.26
v3.0.27
v3.0.28
v3.0.3
v3.0.4
v3.0.5
v3.0.6
v3.0.7
v3.0.8
v3.0.9
v3.1.0
v3.1.0-alpha
v3.1.0-beta
v3.1.0-rc
v3.1.0-rc.2
v3.1.0-rc.3
v3.1.1
v3.1.10
v3.1.11
v3.1.12
v3.1.13
v3.1.14
v3.1.15
v3.1.2
v3.1.3
v3.1.4
v3.1.5
v3.1.6
v3.1.7
v3.1.8
v3.1.9
v3.2-alpha.1
v3.2.0
v3.2.0-alpha.1
v3.2.0-beta
v3.2.0-beta.2
v3.2.0-beta.3
v3.2.1
v3.2.10
v3.2.11
v3.2.12
v3.2.13
v3.2.14
v3.2.15
v3.2.16
v3.2.2
v3.2.3
v3.2.4
v3.2.5
v3.2.6
v3.2.7
v3.2.8
v3.2.9
v4.0.0
v4.0.0-alpha
v4.0.0-beta
v4.0.1
v4.0.2
v4.0.3
v4.0.4
v4.0.5
v4.0.6
v4.0.7
v4.0.8
v4.1.0
v4.1.0-alpha
v4.1.0-beta
v4.1.0-rc
v4.1.1
v4.1.2
v4.1.3
v4.1.4
v4.1.5
Labels
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
Apply labels
Clear labels
accessibilité
amélioration
Amélioration, nouvelle fonctionnalité APIs authentification base de données bug
Ca ne fonctionne pas code généré compilo css divers documentation doublon
Ce ticket est un doublon ergonomie espace privé filtres et balises formulaires Inscription installation invalide
Ticket invalide javascript langues LDAP plugin PostgreSQL refusé
Ignoré, c'est comme Ca... sécurité traduction
No Label
accessibilité
amélioration
APIs
authentification
base de données
bug
code généré
compilo
css
divers
documentation
doublon
ergonomie
espace privé
filtres et balises
formulaires
Inscription
installation
invalide
javascript
langues
LDAP
plugin
PostgreSQL
refusé
sécurité
traduction
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
This issue currently doesn't have any dependencies.
Reference in new issue
There is no content yet.
Delete Branch '%!s(MISSING)'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
Sur un spip 407 ou je suis webmestre, et sur contrib spip 411 où je suis admin, il m'était impossible de changer le mail d'un auteur.
Aucune erreur n'était affichée mais le focus revenait à la saisie de l'email, le form refusant l'enregistrement.
J'ai compris que le problème venait d'un espace au début du mail.
À tester par exemple ici https://contrib.spip.net/ecrire/?exec=auteur_edit&id_auteur=8034 (les adresses @voila sont mortes depuis longtemps donc pas de risque).
J'ai pu modifier l'adresse en question sans problème sur contrib où je suis admin non restreint.
Si le problème signalé est bien l'impossibilité de valider un email contenant un espace en début ou fin de chaîne, je t'invite à reformuler le titre du ticket :)
PS : le problème avec l'espace ne vient pas de la validation côté PHP, mais du fait que l'input est de type email, c'est donc le navigateur qui bloque.
Impossible modifier email d'un auteurto email commençant par un espace est rejeté sans explication 2 months agoC'est un bug côté navigateur donc :
https://bugzilla.mozilla.org/show_bug.cgi?id=849611
https://bugzilla.mozilla.org/show_bug.cgi?id=657702
https://bugzilla.mozilla.org/show_bug.cgi?id=1132142
Et à ce sujet https://jtreminio.com/blog/firefox-input-email-validation/
Ya le pb de pas enlever l'espace avant de tester, maius ya aussi le pb de ne pas afficher de message d'erreur.
En fait, FF propose bien un message d'erreur, mais invisible car totalement masqué (et flushé je crois) par un autre message qui propose "Afficher les identifiants enregistrés" !
Le pb ne se produit pas que lorsqu'il y a une espace en début de mail, mais aussi lorsque le mail est incorrect, par exemple s'il manque le @
Pas du tout, chez moi j'ai bien un message d'erreur affiché par FF juste sous l'input du mail, le message des trucs enregistrés vient du fait que tu utilises le gestionnaire de mot de passe de FF. Bref, tout ça n'a plus grand chose à voir avec SPIP.
une capture d'écran aiderait à comprendre, mais en effet il y a un bug d'interface sous FF, on ne voit pas le message d'erreur du navigateur si on a des identifiants enregistrés sur le site concerné.
Mais donc ça nous concerne un peu, car on devrait pouvoir mettre des attributs qui vont bien pour éviter que les navigateurs ne proposent de remplir l'email ou le login de la page editer_auteur avec les identifiants enregistrés pour ce site vu que c'est pas du tout notre fiche qu'on édite en général...
Mouais bah faut bien tenir compte de comment les navigateurs fonctionnent... Yavait des tweaks pour IE on pourrait imaginer des tweaks pour FF.
Ma bon je ferme.
Pour libérer du temps pour les trucs plus importants !
messages croisés, réouvre stv cerdic
Bon courage, j'ai tenté mais la moindre action ferme la notif du navigateur, la seule solution est de lancer une capture à retadement et tenter de faire afficher la notif au bon moment :D
On le fait déjà je crois bien, avec l'attribut autocomplete off, mais celui-ci est ignoré par les navigateurs cf https://stackoverflow.com/a/468295 & https://stackoverflow.com/a/21348793 => HTML5
rockssucks ^^Voici 2 copies d'écran.
Ça se produit quand on valide avec le bouton Enregistrer : la fenêtre scroll vers le haut jusqu'à l'input email qui reçoit le focus ; je vois maintenant que le message d'erreur apparaît une fraction de seconde mais qu'il est remplacé par l'invitation aux mots de passe.
Quand on fait Enter alors que le focus est dans l'input email, le message d'erreur apparaît bien sans être flushé par l'invitation aux mots de passe.
en fait il faut un
autocomplete="new-password"
sur le champ emailhttps://www.w3docs.com/snippets/html/how-to-disable-browser-autocomplete-and-autofill-on-html-form-and-input-fields.html
... comme on a déjà sur les champs login et password du même formulaire :)
Et donc
#5219
j'ai patché à l'arrache sur contrib pour valider, donc tu dois plus y avoir le problème @JLuc
L’erreur avec l’espace s’affiche très bien chez moi (et reste)
je confirme que c'est réglé sur contrib.
tu arrives après la bataille @marcimat c'est déjà réparé :p