#8038: email address validator does not accept + in addresses
---------------------------------------+------------------------------------
Reporter: ddrake | Owner: ddrake
Type: defect | Status: positive_review
Priority: major | Milestone: sage-4.3.4
Component: notebook | Keywords:
Author: Dan Drake, Mitesh Patel | Upstream: N/A
Reviewer: Tim Dumol | Merged:
Work_issues: |
---------------------------------------+------------------------------------
Changes (by timdumol):
* status: needs_review => positive_review
* reviewer: => Tim Dumol
Comment:
This works well enough. It's not overly complex like this:
{{{
(?:[a-z0-9!#$%&'*+/=?^_`{|}~-]+(?:\.[a-z0-9!#$%&'*+/=?^_`{|}~-]+)*|"(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21\x23-\x5b\x5d-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])*")@(?:(?:[a-z0-9](?:[a-z0-9-]*[a-z0-9])?\.)+[a-z0-9](?:[a-z0-9-]*[a-z0-9])?|\[(?:(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?)\.){3}(?:25[0-5]|2[0-4][0-9]|[01]?[0-9][0-9]?|[a-z0-9-]*[a-z0-9]:(?:[\x01-\x08\x0b\x0c\x0e-\x1f\x21-\x5a\x53-\x7f]|\\[\x01-\x09\x0b\x0c\x0e-\x7f])+)\])
}}}
which still does not cover the new IPv6 addresses (c.f.
http://snipplr.com/view/20981/rfc-2822-email-validation/). Doctests pass,
LGTM.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/8038#comment:18>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en.
