Hi Yasuo, I've not changed (and even read) the email validator. I'll take a look at it.
2014-11-12 10:41 GMT+01:00 Yasuo Ohgaki <yohg...@ohgaki.net>: > Hi Kevin, > > On Wed, Nov 12, 2014 at 4:09 PM, Kévin Dunglas <dung...@gmail.com> wrote: > >> I'll change my PR according to the RFC I've quoted earlier: >> - check for valid characters (excluding underscore) only when >> FILTER_FLAG_HOSTNAME is set >> - allow any character but check lengths by default >> - use FILTER_FLAG_HOSTNAME to validate URLs >> >> What do you think about that? >> > > I haven't read diff closely, but it seems ok to me. > How email domain is checked? I cannot see changes for it from the diff. > > Validating host correctly is difficult, I would like to have your PR. > > Regards, > > -- > Yasuo Ohgaki > yohg...@ohgaki.net > -- Kévin Dunglas http://dunglas.fr
