php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Doc Bug #17533 parse_url shouldn't work with http://www..com
Submitted: 2002-05-30 17:53 UTC Modified: 2002-06-09 19:37 UTC
From: dd at selftrade dot com Assigned:
Status: Closed Package: Documentation problem
PHP Version: 4.2.1 OS: RedHat 7.2
Private report: No CVE-ID: None
View Add Comment Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
You can add a comment by following this link or if you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: dd at selftrade dot com
New email:
PHP Version: OS:

 

 [2002-05-30 17:53 UTC] dd at selftrade dot com 
If you submit a url like www..com to parse_url, it will parse it without any error, although this kind of URL is incorrect.

Try this:

<?

$url="http://www..com";

$a=parse_url($url);

$host=$a[host];
$scheme=$a[scheme];
$path=$a[path];

echo("<LI>SCHEME=$scheme<LI>HOST=$host<LI>PATH=$path");

?>

It will return:

SCHEME=http
HOST=www..com
PATH=

The same kind of thing happens if url="qwerty", as it will return "qwerty" as the path.

Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2002-06-04 14:20 UTC] mfischer@php.net 
Just a note:

"qwerty" on it's own still can be right (e.g. you have a subdomain qwerty.mydomain.com and set up your box that mydomain.com is in your domain search path).

On the other hand:
php -r 'print_r(parse_url("http://www][.ww_-/...._..-.../"));'

Array
(
    [scheme] => http
    [host] => www][.ww_-
    [path] => /...._..-.../
)

So it looks like parse_url() only splits the url into it's distinct parts (which obviously works).

If this is true, this needs to be documented. Anyone else comments?
 [2002-06-04 16:15 UTC] mfischer@php.net 
After examinign the code:

parse_url is not meant to validate the URL, just to break it up into it's distinct parts if possible.

Reclassifying as documentation problem.
 [2002-06-09 19:37 UTC] mfischer@php.net 
This bug has been fixed in CVS. You can grab a snapshot of the
CVS version at http://snaps.php.net/. In case this was a documentation 
problem, the fix will show up soon at http://www.php.net/manual/.
In case this was a PHP.net website problem, the change will show
up on the PHP.net site and on the mirror sites.
Thank you for the report, and for helping us make PHP better.
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved. 		Last updated: Fri Apr 19 20:01:29 2024 UTC