#167 MailingList.class : top domain limit

enhancement
closed-accepted
nobody
5
2003-01-23
2002-12-16
tobozo
No

Hi,

The expression in the method is_valid_email() from the
MailingList class needs some update to handle top
domain names like '.info' or '.museum'.
It is actually limited to top domains with 2 or 3 letters
only.

Suggestion for fix :
if(eregi("([_\.0-9a-z-]+@)([0-9a-z][0-9a-z-]+\.)+([a-z]
{2,6})", $address))

be well

tobozo

Discussion

  • tobozo
    tobozo
    2002-12-16

    Logged In: YES
    user_id=126727

    oops ... should 2,7 and not 2,6 (didn'd read the ICANN
    update hehe) ... but the regex can still be improved :
    http://regexlib.com/REDetails.aspx?regexp_id=192

     
  • Joe Stewart
    Joe Stewart
    2003-01-23

    • status: open --> closed-accepted