Produits Entr'ouvert » Authentic 2

gmail.fr, gmail.cm, hotnail.fr ... passent les tests mécaniques de l'existence d'un MX ou d'un A. La vérification du domaine n'est pas la bonne piste.

Frédéric montrait par ailleurs https://github.com/mailcheck/mailcheck : cela suggère (via un poil de jajascript) à l'usager qu'il a peut-être fait une erreur, sans autre intelligence que la proximité avec des noms de services bien connus. C'est nettement plus pertinent à mon avis.

Ça existe déjà, option A2_VALIDATE_EMAIL_DOMAIN (qui est à True) et A2_VALIDATE_EMAIL (qui fait un RCPT et est à False par défaut), mais le validateur contrôlé par ça n'est pas utilisé sur RegistrationForm (authentic2.validators.EmailValidator).

Pour le reste je suggère effectivement 30 lignes de JS custom avec les 30 domaines qui nous intéressent et une implémentation basique d'une distance d'édition entre chaîne (genre levenshtein¹ ou autre plus simple²) plutôt qu'une dépendance externe pour un truc aussi simple.

¹ https://gist.github.com/andrei-m/982927

² https://en.wikipedia.org/wiki/Edit_distance

Serghei Mihai a écrit :

Benjamin Dauvergne a écrit :

Ça existe déjà, option A2_VALIDATE_EMAIL_DOMAIN (qui est à True) et A2_VALIDATE_EMAIL (qui fait un RCPT et est à False par défaut), mais le validateur contrôlé par ça n'est pas utilisé sur RegistrationForm (authentic2.validators.EmailValidator).

Il n'est pas utilisé du tout ce validateur. Je fais un ticket pour s'en servir dans le formulaire d'enregistrement, même si pas de rapport avec la direction prise dans ce ticket.

Il me semble qu'il l'est sur le champ email du modèle, c'est juste que les modifications du champ email ayant été bougé dans leur propre form progressivement ça n'a pas été transporté.

Au cas où, j'ai ça qui est en cours côté w.c.s. (#42396), si c'est important ici je peux le terminer là.

0001-scss-reduce-field-s-live-hint-button-min-width-40166.patch

Visiblement mal indenté.

    A2_SUGGESTED_EMAIL_DOMAINS=Setting(
        default=[],

On pourrait avoir un comportement utile par défaut, plutôt qu'en faire un élément en plus qu'il faudra configurer, ou un élément en plus qui ne sera jamais configuré.

Concernant les textes, il faudrait ceux qui ont été discutés dans le ticket w.c.s. ("Did you want to write" et "Apply fix").

Pour référence, le min-width, c'est pour contrer :

div.a2-block form button, div.a2-block form input[type="submit"] {
    width: auto;
    min-width: 12em;
}

(je suggérerais de plutôt mettre min-width: auto, là j'ai cru avant de regarder le code que c'était pour assurer une largeur minimale, ce qui me surprenait vu que le texte devait déjà assurer celle-ci).

La popup apparait brièvement lors du rendu de la page; je pense qu'il faut ajouter style="display: none" dans le gabarit.

Ça n'est pas effectif en backoffice. (?)