php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
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
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
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: Tue Apr 16 19:01:31 2024 UTC