Edit report at http://bugs.php.net/bug.php?id=51192&edit=1
ID: 51192
Comment by: ond...@php.net
Reported by: solar at azrael dot ws
Summary: FILTER_VALIDATE_URL will invalidate a hostname that
includes '-'
Status: Closed
Type: Bug
Package: Filter related
Operating System: linux
PHP Version: 5.2.13
Assigned To: aharvey
Block user comment: N
New Comment:
Please be aware that IDN (PunyCode) url can contain two consecutive
hyphens, ie.
xn--hkyrky-ptac70bc.cz
is OK as well.
Previous Comments:
------------------------------------------------------------------------
[2010-04-07 02:43:39] ahar...@php.net
The fix for this hasn't been incorporated into any stable releases yet:
it came after 5.2.13 was released and 5.3.2 was branched (and far too
late in 5.3.2's release cycle to be ported across).
Both versions 5.2.14 and 5.3.3 will include this bug fix.
------------------------------------------------------------------------
[2010-04-06 22:04:34] lupo at e-noise dot com
I am experiencing the same behavior with PHP 5.3.2 on Mac OSX. When
hostname contains hyphens filter_var() won't validate using the
FILTER_VALIDATE_URL filter.
The test code posted by "solar at azrael dot ws" produces the following
output:
string(18) "http://example.com";
bool(false)
string(19) "http://exa_mple.com";
------------------------------------------------------------------------
[2010-03-03 10:26:58] ahar...@php.net
This bug has been fixed in SVN.
Snapshots of the sources are packaged every three hours; this change
will be in the next snapshot. You can grab the snapshot at
http://snaps.php.net/.
Thank you for the report, and for helping us make PHP better.
------------------------------------------------------------------------
[2010-03-03 10:25:52] ahar...@php.net
Automatic comment from SVN on behalf of aharvey
Revision: http://svn.php.net/viewvc/?view=revision&revision=295773
Log: Fix for bug #51192 (FILTER_VALIDATE_URL will invalidate a hostname
that includes '-'). Original patch by so...@azrael.ws.
------------------------------------------------------------------------
[2010-03-03 09:50:24] ahar...@php.net
Yeah, that doesn't quite look right. :)
Cheers for the patch; I'll cook up a test script to go with it and
commit it.
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://bugs.php.net/bug.php?id=51192
--
Edit this bug report at http://bugs.php.net/bug.php?id=51192&edit=1
