Description: ------------ if you have a meta refresh that looks like this: <meta http-equiv="refresh" content="0; url='/test/httpclientmeta.html'"> located on this server: http://gemal.dk/test/httpclientmeta.html then the meta refresh is going to be matches as: Array ( [0] => <meta http-equiv="refresh" content="0; url='/test/httpclientmeta.html'"> [1] => http-equiv="refresh" content="0; url='/test/httpclientmeta.html'" [2] => "refresh" ) ret=http://gemal.dk/test/'/test/httpclientmeta.html' note the ' in the url!!!

changing line 573 to: if (empty($parts[1]) || !preg_match('/url\\s*=\\s*\'?(\\S+)(?<!\')/is', $parts[1], $urlMatches)) { seems to fix the problem

should I produce a patch to get this one fixed?

Henrik, that's certainly a good idea.

I'm unable to attach the patch as a file due to insufficient priviliages so: Index: Client.php =================================================================== RCS file: /repository/pear/HTTP_Client/Client.php,v retrieving revision 1.8 diff -u -i -w -r1.8 Client.php --- Client.php 19 May 2007 14:57:31 -0000 1.8 +++ Client.php 25 May 2008 11:45:13 -0000 @@ -570,7 +570,7 @@ } $parts = explode(';', ('\'' == substr($urlMatches[1], 0, 1) || '"' == substr($urlMatches[1], 0, 1))? substr($urlMatches[1], 1, -1): $urlMatches[1]); - if (empty($parts[1]) || !preg_match('/url\\s*=\\s*(\\S+)/is', $parts[1], $urlMatches)) { + if (empty($parts[1]) || !preg_match('/url\\s*=\\s*\'?(\\S+)(?<!\')/is', $parts[1], $urlMatches)) { return null; } // We do finally have an url... Now check that it's:

I have attached a patch but nothing happend :(

Looks like that it can also be <meta http-equiv='refresh' content='0; url="/foo.html"'> Will have to address all cases.

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.