@robocoder opened this Issue on October 22nd 2010 Contributor

Piwik::isValidEmailString could be modified to comply with rfc5321:

   Mailbox        = Local-part "@" ( Domain / address-literal )

   Local-part     = Dot-string / Quoted-string
                  ; MAY be case-sensitive

   Dot-string     = Atom *("."  Atom)

   Atom           = 1*atext

   Quoted-string  = DQUOTE *QcontentSMTP DQUOTE

   QcontentSMTP   = qtextSMTP / quoted-pairSMTP

   quoted-pairSMTP  = %d92 %d32-126
                    ; i.e., backslash followed by any ASCII
                    ; graphic (including itself) or SPace

   qtextSMTP      = %d32-33 / %d35-91 / %d93-126
                  ; i.e., within a quoted string, any
                  ; ASCII graphic or space is permitted
                  ; without blackslash-quoting except
                  ; double-quote and the backslash itself.

   atext           =       ALPHA / DIGIT / ; Any character except controls,
                        "!" / "#" /     ;  SP, and specials.
                        "$" / "%" /     ;  Used for atoms
                        "&" / "'" /
                        "*" / "+" /
                        "-" / "/" /
                        "=" / "?" /
                        "^" / "_" /
                        "`" / "{" /
                        "|" / "}" /
                        "~"
@mattab commented on November 16th 2010 Member

is there a problem in current function?

@robocoder commented on November 16th 2010 Contributor

The current implementation is strict. The RFC is very permissive.

I was originally thinking we make this configureable via a regex in global.ini.php, but instead, maybe we can use Zend_Validate_EmailAddress?

@mattab commented on November 24th 2010 Member

let's just find & copy an existing RFC compliant email test function (or reuse zend if it's not too bloated...)

@mattab commented on April 28th 2011 Member

we can reopen if this poses problem so far so good

This Issue was closed on April 28th 2011
Powered by GitHub Issue Mirror