[Django] #27932: EmailValidator fails on "a.@example.com"
Django
-----------------------------------------+------------------------
Reporter: Aur Saraf | Owner: nobody
Type: Uncategorized | Status: new
Component: Uncategorized | Version: 1.10
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 |
-----------------------------------------+------------------------
This is a perfectly valid email address, and I found it because a user of
mine tried to register with a similar address.
I confirmed that I can send email to `aur.@feezback.com`.
--
Ticket URL: <https://code.djangoproject.com/ticket/27932>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.
Django
Reporter: Aur Saraf | Owner: MisRob
Type: Uncategorized | Status: assigned
Component: Uncategorized | Version: 1.10
Severity: Normal | Resolution:
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 MisRob):
* owner: nobody => MisRob
* status: new => assigned
--
Ticket URL: <https://code.djangoproject.com/ticket/27932#comment:1>
Django
Reporter: Aur Saraf | Owner: Michaela
| Robosova
Type: Uncategorized | Status: closed
Component: Uncategorized | Version: 1.10
Severity: Normal | Resolution: invalid
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 Michaela Robosova):
* status: assigned => closed
* resolution: => invalid
Comment:
I would say it is perfectly fine that this e-mail didn't pass through
e-mail validation because dot character is not allowed as the last
character of the email local-part.
Pls. see https://tools.ietf.org/html/rfc5322, sections 3.4.1, 3.2.3, 3.2.4
and 4.4.
Maybe some mail servers still allow this obsolete syntax but this fact
surely doesn't mean EmailValidator should accept it.
--
Ticket URL: <https://code.djangoproject.com/ticket/27932#comment:2>
Django
Reporter: Aur Saraf | Owner: Michaela
| Robosova
Type: Uncategorized | Status: closed
Severity: Normal | Resolution: invalid
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 Michaela Robosova):
* cc: robosova.michaela@… (added)
* component: Uncategorized => Core (Mail)
--
Ticket URL: <https://code.djangoproject.com/ticket/27932#comment:3>
Django
Reporter: Aur Saraf | Owner: Michaela
| Robosova
Type: Uncategorized | Status: closed
Component: Core (Mail) | Version: 1.10
Severity: Normal | Resolution: invalid
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Comment (by Tim Graham):
By the way, #26423 will change `EmailValidator` to use HTML5 validation.
That validation is more liberal and may accept this address.
--
Ticket URL: <https://code.djangoproject.com/ticket/27932#comment:4>
Django
Reporter: Aur Saraf | Owner: MisRob
Component: Core (Mail) | Version: 1.10
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 MisRob):
* resolution: invalid => duplicate
Comment:
Oh, that's a good point! I had no idea about this, sorry. It seems to be a
nice idea to simplify the validation. Anyone can use custom regex if
needed. So I guess it will be better to close this one as duplicate to
#26423.
--
Ticket URL: <https://code.djangoproject.com/ticket/27932#comment:5>
Django
Reporter: Aur Saraf | Owner: MisRob
Cleanup/optimization |
Component: Core (Mail) | Version: 1.10
Severity: Normal | Resolution: duplicate
| Unreviewed
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
Changes (by MisRob):
* type: Uncategorized => Cleanup/optimization
--
Ticket URL: <https://code.djangoproject.com/ticket/27932#comment:6>