View Issue Details

IDProjectCategoryView StatusLast Update
0015632phplist applicationEmailpublic23-05-12 03:51
ReporterAndy Read 
PrioritynormalSeverityblockReproducibilityalways
Status resolvedResolutionfixed 
Product Version2.10.14 
Target Version4.0.xFixed in Version2.11.7 
Summary0015632: TLDs missing from is_email() validation
DescriptionThe 'im' TLD (Isle of Mann) is missing from the is_email() validation function in admin/commonlib/lib/userlib.php.

A cursory look at the pattern used suggests that several other TLDs are missing compared to http://en.wikipedia.org/wiki/List_of_Internet_top-level_domains.

e.g. .mobi, .tel, .xxx, not to mention internationalized and the new "register-anything-you-like" (if you can afford it) TLDs.
Additional InformationI suggest this is_email() function needs to be written to use an external txt file of TLDs. At a minimum this could be updated from the official http://data.iana.org/TLD/tlds-alpha-by-domain.txt at each new release of phplist and would be fairly easy to edit by admins if necessary. Ideally phplist would check itself for updated versions of this file.
TagsNo tags attached.

Relationships

has duplicate 0011959 resolvedmichiel Missing TLD for .travel domains 
related to 0012866 resolvedmichiel Strict format checking of email and other fields prevents numerous imports, possible for mysql injection attacks 
related to 0006782 resolvedmichiel Importing Invalid E-mails 

Activities

michiel

20-09-11 14:44

manager   ~0051405


"im" is back in the regex, but the issue of loading the list from a remote source should be looked at for new dev

michiel

22-05-12 20:18

manager   ~0051582

I've added a dynamic update of TLDs