[Django] #31959: EmailValidator must not accept hyphen as first caracter

17 views
Skip to first unread message

Django

unread,
Aug 28, 2020, 7:15:13 AM8/28/20
to django-...@googlegroups.com
#31959: EmailValidator must not accept hyphen as first caracter
-----------------------------------------+------------------------
Reporter: James Pic | Owner: nobody
Type: Uncategorized | Status: new
Component: Utilities | Version: 3.1
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-----------------------------------------+------------------------
Getting the following exception:

Aug 28 10:57:22 production 83a5b18c66bc[1752]:
smtplib.SMTPRecipientsRefused: {'-reda...@laposte.net': (501, b'5.1.3 Bad
recipient address syntax')}

EmailValidator should not accept a hyphen as first caracter.

Note: Component set to "Utilities" but really I believe a "Validator"
Component should be available.

--
Ticket URL: <https://code.djangoproject.com/ticket/31959>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

Django

unread,
Aug 28, 2020, 7:51:38 AM8/28/20
to django-...@googlegroups.com
#31959: EmailValidator must not accept hyphen as first character
-------------------------------------+-------------------------------------

Reporter: James Pic | Owner: nobody
Type: | Status: closed
Cleanup/optimization |
Component: Utilities | Version: 3.1
Severity: Normal | Resolution: wontfix

Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by felixxm):

* status: new => closed
* type: Uncategorized => Cleanup/optimization
* resolution: => wontfix


Comment:

Thanks for this ticket, however we discussed a similar case in #25452 and
the consensus on the [https://groups.google.com/d/topic/django-
developers/ASBJ0ge2KYo/discussion mailing list] was to simplify the
validation, not make it more comprehensive.

--
Ticket URL: <https://code.djangoproject.com/ticket/31959#comment:1>

Django

unread,
Aug 28, 2020, 11:53:11 AM8/28/20
to django-...@googlegroups.com
#31959: EmailValidator must not accept hyphen as first character
-------------------------------------+-------------------------------------

Reporter: James Pic | Owner: nobody
Type: | Status: closed
Cleanup/optimization |
Component: Utilities | Version: 3.1
Severity: Normal | Resolution: wontfix

Keywords: | Triage Stage:
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by James Pic):

Thanks to you Felix, I now know that EmailField is deliberately broken and
untrustable.

However, this needs to be properly documented, and inaccuracies in the
documentation must be fixed.

Following-up on this ticket : https://code.djangoproject.com/ticket/31960

--
Ticket URL: <https://code.djangoproject.com/ticket/31959#comment:2>

Reply all
Reply to author
Forward
0 new messages