[Django] #22123: EmailValidator does not accept IPv6 literals

12 views
Skip to first unread message

Django

unread,
Feb 22, 2014, 9:54:41 AM2/22/14
to django-...@googlegroups.com
#22123: EmailValidator does not accept IPv6 literals
----------------------------+------------------------
Reporter: erikr | Owner: nobody
Type: Bug | Status: new
Component: Forms | Version: master
Severity: Normal | Keywords: nlsprint14
Triage Stage: Unreviewed | Has patch: 0
Easy pickings: 0 | UI/UX: 0
----------------------------+------------------------
EmailValidator currently supports normal domains (f...@example.com) and
IPv4 literals (foo@[192.0.2.1]). IPv6 support is currently missing
(foo@[2001:db8::1]). This is specified to be valid, along with the other
IPv6 notations, in RFC 5321 section 4.1.3.

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

Django

unread,
Feb 22, 2014, 10:01:18 AM2/22/14
to django-...@googlegroups.com
#22123: EmailValidator does not accept IPv6 literals
----------------------------+--------------------------------------

Reporter: erikr | Owner: nobody
Type: Bug | Status: new
Component: Forms | Version: master
Severity: Normal | Resolution:

Keywords: nlsprint14 | Triage Stage: Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
----------------------------+--------------------------------------
Changes (by erikr):

* cc: eromijn@… (added)
* needs_better_patch: => 0
* has_patch: 0 => 1
* needs_tests: => 0
* needs_docs: => 0


Comment:

PR in https://github.com/django/django/pull/2348

This also cleans up the IPv4 regex used in EmailValidator, as that is
already available as part of the existing IPv4 validator. The regex
previously used as part of domain_regex was identical, so this does not
break backwards compatibility.

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

Django

unread,
Feb 22, 2014, 10:57:55 AM2/22/14
to django-...@googlegroups.com
#22123: EmailValidator does not accept IPv6 literals
----------------------------+--------------------------------------
Reporter: erikr | Owner: erikr
Type: Bug | Status: assigned
Component: Forms | Version: master
Severity: Normal | Resolution:

Keywords: nlsprint14 | Triage Stage: Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
----------------------------+--------------------------------------
Changes (by erikr):

* status: new => assigned
* owner: nobody => erikr


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

Django

unread,
Feb 22, 2014, 12:10:28 PM2/22/14
to django-...@googlegroups.com
#22123: EmailValidator does not accept IPv6 literals
----------------------------+--------------------------------------
Reporter: erikr | Owner: erikr
Type: Bug | Status: closed
Component: Forms | Version: master
Severity: Normal | Resolution: fixed

Keywords: nlsprint14 | Triage Stage: Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
----------------------------+--------------------------------------
Changes (by Baptiste Mispelon <bmispelon@…>):

* status: assigned => closed
* resolution: => fixed


Comment:

In [changeset:"98f13762d7e18c0ba12044eed56e9198ca056c66"]:
{{{
#!CommitTicketReference repository=""
revision="98f13762d7e18c0ba12044eed56e9198ca056c66"
Fixed #22123 -- EmailValidator now also accepts IPv6 literals in addresses
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/22123#comment:3>

Django

unread,
Aug 6, 2021, 6:59:40 AM8/6/21
to django-...@googlegroups.com
#22123: EmailValidator does not accept IPv6 literals
------------------------------+----------------------------------------
Reporter: Sasha Romijn | Owner: Sasha Romijn
Type: Bug | Status: closed
Component: Forms | Version: dev
Severity: Normal | Resolution: fixed

Keywords: nlsprint14 | Triage Stage: Unreviewed
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0

Easy pickings: 0 | UI/UX: 0
------------------------------+----------------------------------------

Comment (by Mariusz Felisiak <felisiak.mariusz@…>):

In [changeset:"61d92c650f5bc363177ebace62e3e8a67ec9499d" 61d92c6]:
{{{
#!CommitTicketReference repository=""
revision="61d92c650f5bc363177ebace62e3e8a67ec9499d"
Refs #22123 -- Added more URLValidator test for invalid IPv6 literals.
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/22123#comment:4>

Reply all
Reply to author
Forward
0 new messages