On 19 Sep 2014, at 06:52, Andrea Faulds <a...@ajf.me> wrote: > > On 18 Sep 2014, at 21:26, Kévin Dunglas <dung...@gmail.com> wrote: > >> I'm working on enhancing the FILTER_VALIDATE_URL filter ( >> https://github.com/php/php-src/pull/826). >> The current implementation does not support validation of internationalized >> domain names (i.e: http://www.académie-française.fr/ >> <http://www.xn--acadmie-franaise-npb1a.fr/>). >> >> Support of IDN validation can be easily added using ICU's uidna_toASCII() >> function. >> >> Is it acceptable to add a dependency to ICU for ext/filter? >> Another option is to add a HAVE_ICU constant in main/php_config.h and to >> validate IDN only if ICU is present. >> >> What strategy is preferred? > > Perhaps add a new filter that covers normal URLs and IDN ones? I just imagine > it might cause problems if suddenly IDNs are accepted, if there is a backend > which can’t handle them.
We don’t need a new filter, you can simply add a filter flag for FILTER_VALIDATE_URL, e.g. FILTER_FLAG_ALLOW_IDN. Of course, the ICU dependency should be optional :) > > -- > Andrea Faulds > http://ajf.me/ > > > > > > -- > PHP Internals - PHP Runtime Development Mailing List > To unsubscribe, visit: http://www.php.net/unsub.php > -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php