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

Reply via email to