@anonymous-piwik-user opened this Issue on July 4th 2011

Currently the TLD .travel is not supported under e-mail reports. When trying to add additional e-mail addresses for "Send report to:" Piwik gives the following error: "The email doesn't have a valid format. (person@domain.travel)"

Please could you add .travel to the TLD array.


@halfdan commented on July 4th 2011 Member

This is quite easy to fix, in core/Piwik.php:

return (preg_match('/^[a-zA-Z0-9_.+-]+@[a-zA-Z0-9_.-]+\.[a-zA-Z]{2,4}$/', $email) > 0);

should actually be

return (preg_match('/^[a-zA-Z0-9_.+-]+@[a-zA-Z0-9_.-]+\.[a-zA-Z]{2,6}$/', $email) > 0);

.museum/.travel and others are currently not accepted as valid TLDs by the regex above.

@robocoder commented on July 4th 2011 Contributor

We also have internationalized tlds to address in #1755. Maybe address boyh tickeys at the same time, eg using ZF hostname validator.

@mattab commented on September 27th 2011 Member

(In [5239]) Fixes #2535

This Issue was closed on September 27th 2011
Powered by GitHub Issue Mirror