Description: ------------ When $_SERVER['PHP_SELF'] ends on /, HTTP::absoluteURI() calling dirname($_SERVER['PHP_SELF']) results in everything after the leading / getting chopped off, which causes incorrect redirects for relative URIs. That attached patch against HEAD fixes this behaviour by not touching PHP_SELF when it comes with a trailing slash. Test script: --------------- Assuming $_SERVER['PHP_SELF'] is /foobar/. <?php require_once "HTTP.php"; HTTP::redirect("page.html"); ?> Expected result: ---------------- A redirect to http://myhost.tld/foobar/page.html. Actual result: -------------- A redirect to http://myhost.tld/page.html.

The HTTP package trims PATH_INTO (in your example "/" or "/foobar/") to resolves relative URLs to the script itself. The patch cannot be accepted because it would cause a backward compatibility break. I'm working on solution to handle redirection taking into account the PATH_INFO, REQUEST_URI and generally friendlier to rewrite rules.

Thank you for taking the time to report a problem with the package. This problem may have been already fixed by a previous change that is in the CVS of the package. Please log into CVS with: cvs -d :pserver:cvsread@cvs.php.net:/repository login and check out the CVS repository of this package and upgrade cvs -d :pserver:cvsread@cvs.php.net:/repository co pear/HTTP pear upgrade pear/HTTP/package2.xml or pear upgrade pear/HTTP/package.xml If you are able to reproduce the bug with the latest CVS, please change the status back to "Open". Again, thank you for your continued support of PEAR.