diff options
-rw-r--r-- | doc/ChangeLog | 5 | ||||
-rw-r--r-- | src/SemanticScuttle/Environment.php | 26 | ||||
-rw-r--r-- | src/SemanticScuttle/constants.php | 4 | ||||
-rw-r--r-- | tests/SemanticScuttle/EnvironmentTest.php | 139 |
4 files changed, 157 insertions, 17 deletions
diff --git a/doc/ChangeLog b/doc/ChangeLog index ff60527..ca26618 100644 --- a/doc/ChangeLog +++ b/doc/ChangeLog @@ -3,6 +3,11 @@ ChangeLog for SemantiScuttle .. contents:: +0.98.6 - 2013-XX-XX +------------------- +- Fix bug #161: URLs broken on 1&1 server + + 0.98.5 - 2013-03-20 ------------------- - Fix bug #109: preserve privacy setting from Delicious export files diff --git a/src/SemanticScuttle/Environment.php b/src/SemanticScuttle/Environment.php index e5fe3de..7ccb466 100644 --- a/src/SemanticScuttle/Environment.php +++ b/src/SemanticScuttle/Environment.php @@ -29,24 +29,20 @@ class SemanticScuttle_Environment */ public static function getServerPathInfo() { - /* old code that does not work today. - if you find that this code helps you, tell us - and send us the output of var_export($_SERVER); - // Correct bugs with PATH_INFO (maybe for Apache 1 or CGI) -- for 1&1 host... - if (isset($_SERVER['PATH_INFO']) && isset($_SERVER['ORIG_PATH_INFO'])) { - if (strlen($_SERVER["PATH_INFO"])<strlen($_SERVER["ORIG_PATH_INFO"])) { - $_SERVER["PATH_INFO"] = $_SERVER["ORIG_PATH_INFO"]; - } - if (strcasecmp($_SERVER["PATH_INFO"], $_SERVER["SCRIPT_NAME"]) == 0) { - unset($_SERVER["PATH_INFO"]); - } - if (strpos($_SERVER["PATH_INFO"], '.php') !== false) { - unset($_SERVER["PATH_INFO"]); + if (isset($_SERVER['PATH_INFO'])) { + return $_SERVER['PATH_INFO']; + } + + if (isset($_SERVER['ORIG_PATH_INFO'])) { + //1&1 servers + if ($_SERVER['ORIG_PATH_INFO'] == $_SERVER['SCRIPT_NAME']) { + return ''; } + return $_SERVER['ORIG_PATH_INFO']; } - */ - return $_SERVER['PATH_INFO']; + //fallback when no special path after the php file is given + return ''; } } ?>
\ No newline at end of file diff --git a/src/SemanticScuttle/constants.php b/src/SemanticScuttle/constants.php index fcb2d90..306c32a 100644 --- a/src/SemanticScuttle/constants.php +++ b/src/SemanticScuttle/constants.php @@ -69,6 +69,6 @@ define('PAGE_WATCHLIST', "watchlist"); // installations on the same host server define('INSTALLATION_ID', md5($GLOBALS['dbname'].$GLOBALS['tableprefix'])); -//currently not needed -//$_SERVER['PATH_INFO'] = SemanticScuttle_Environment::getServerPathInfo(); +//fix PATH_INFO on certain hosts +$_SERVER['PATH_INFO'] = SemanticScuttle_Environment::getServerPathInfo(); ?> diff --git a/tests/SemanticScuttle/EnvironmentTest.php b/tests/SemanticScuttle/EnvironmentTest.php index a41efa1..3baa9ed 100644 --- a/tests/SemanticScuttle/EnvironmentTest.php +++ b/tests/SemanticScuttle/EnvironmentTest.php @@ -2,6 +2,42 @@ class SemanticScuttle_EnvironmentTest extends PHPUnit_Framework_TestCase { + public function testServerPathInfoModPhpNoPath() + { + $_SERVER = array ( + 'HTTP_USER_AGENT' => 'Opera/9.80 (X11; Linux x86_64) Presto/2.12.388 Version/12.16', + 'HTTP_HOST' => 'bm.bogo', + 'HTTP_ACCEPT' => 'text/html, application/xml;q=0.9, applicaton/xhtml+xml, image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1', + 'HTTP_ACCEPT_LANGUAGE' => 'en,de-DE;q=0.9,de;q=0.8', + 'HTTP_ACCEPT_ENCODING' => 'gzip, deflate', + 'HTTP_CONNECTION' => 'Keep-Alive', + 'HTTP_DNT' => '1', + 'PATH' => '/usr/local/bin:/usr/bin:/bin', + 'SERVER_SIGNATURE' => '<address>Apache/2.2.22 (Ubuntu) Server at bm.bogo Port 80</address>', + 'SERVER_SOFTWARE' => 'Apache/2.2.22 (Ubuntu)', + 'SERVER_NAME' => 'bm.bogo', + 'SERVER_ADDR' => '127.0.0.1', + 'SERVER_PORT' => '80', + 'REMOTE_ADDR' => '127.0.0.1', + 'DOCUMENT_ROOT' => '/var/www', + 'SERVER_ADMIN' => '[no address given]', + 'SCRIPT_FILENAME' => '/home/cweiske/Dev/html/hosts/bm.bogo/test.php', + 'REMOTE_PORT' => '38545', + 'GATEWAY_INTERFACE' => 'CGI/1.1', + 'SERVER_PROTOCOL' => 'HTTP/1.1', + 'REQUEST_METHOD' => 'GET', + 'QUERY_STRING' => '', + 'REQUEST_URI' => '/test.php', + 'SCRIPT_NAME' => '/test.php', + 'PHP_SELF' => '/test.php', + 'REQUEST_TIME_FLOAT' => 1377024570.296, + 'REQUEST_TIME' => 1377024570, + ); + $this->assertEquals( + '', SemanticScuttle_Environment::getServerPathInfo() + ); + } + public function testServerPathInfoModPhp() { $_SERVER = array( @@ -90,6 +126,109 @@ class SemanticScuttle_EnvironmentTest extends PHPUnit_Framework_TestCase ); } + public function testServerPathInfo1and1NoPath() + { + $_SERVER = array( + 'REDIRECT_SCRIPT_URL' => '/dummy.php', + 'REDIRECT_SCRIPT_URI' => 'http://www.example.org/dummy.php', + 'REDIRECT_DOCUMENT_ROOT' => '/kunden/homepages/44/dexample/htdocs/example/www', + 'REDIRECT_HANDLER' => 'x-mapp-php6', + 'REDIRECT_STATUS' => '200', + 'DBENTRY_HOST' => 'example.org', + 'DBENTRY' => '/kunden/homepages/44/dexample/htdocs/example/www:d0000#CPU 6 #MEM 10240 #CGI 18 #NPROC 12 #TAID 46322755 #WERB 0 #LANG 2 #STAT 1', + 'SCRIPT_URL' => '/dummy.php', + 'SCRIPT_URI' => 'http://www.example.org/dummy.php', + 'HTTP_USER_AGENT' => 'Opera/9.80 (X11; Linux x86_64) Presto/2.12.388 Version/12.16', + 'HTTP_HOST' => 'www.example.org', + 'HTTP_ACCEPT' => 'text/html, application/xml;q=0.9, application/xhtml+xml, image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1', + 'HTTP_ACCEPT_LANGUAGE' => 'en,de-DE;q=0.9,de;q=0.8', + 'HTTP_ACCEPT_ENCODING' => 'gzip, deflate', + 'HTTP_COOKIE' => 'PHPSESSID=8c7853d7f639b3c6d24c224cf7d4cb1c', + 'HTTP_CONNECTION' => 'Keep-Alive', + 'HTTP_DNT' => '1', + 'PATH' => '/bin:/usr/bin', + 'SERVER_SIGNATURE' => '', + 'SERVER_SOFTWARE' => 'Apache', + 'SERVER_NAME' => 'example.org', + 'SERVER_ADDR' => '127.0.0.1', + 'SERVER_PORT' => '80', + 'REMOTE_ADDR' => '127.0.0.1', + 'DOCUMENT_ROOT' => '/kunden/homepages/44/dexample/htdocs/example/www', + 'SERVER_ADMIN' => 'webmaster@example.org', + 'SCRIPT_FILENAME' => '/kunden/homepages/44/dexample/htdocs/example/www/dummy.php', + 'REMOTE_PORT' => '35368', + 'REDIRECT_URL' => '/dummy.php', + 'GATEWAY_INTERFACE' => 'CGI/1.1', + 'SERVER_PROTOCOL' => 'HTTP/1.1', + 'REQUEST_METHOD' => 'GET', + 'QUERY_STRING' => '', + 'REQUEST_URI' => '/dummy.php', + 'SCRIPT_NAME' => '/dummy.php', + 'STATUS' => '200', + 'ORIG_PATH_INFO' => '/dummy.php', + 'ORIG_PATH_TRANSLATED' => '/kunden/homepages/44/dexample/htdocs/example/www/dummy.php', + 'PHP_SELF' => '/dummy.php', + 'REQUEST_TIME_FLOAT' => 1377022156.0101, + 'REQUEST_TIME' => 1377022156, + 'argv' => array(), + 'argc' => 0, + ); + $this->assertEquals( + '', SemanticScuttle_Environment::getServerPathInfo() + ); + } + + public function testServerPathInfo1and1WithPath() + { + $_SERVER = array( + 'REDIRECT_SCRIPT_URL' => '/dummy.php/dummy/foo', + 'REDIRECT_SCRIPT_URI' => 'http://www.example.org/dummy.php/dummy/foo', + 'REDIRECT_DOCUMENT_ROOT' => '/kunden/homepages/44/dexample/htdocs/example/www', + 'REDIRECT_HANDLER' => 'x-mapp-php6', + 'REDIRECT_STATUS' => '200', + 'DBENTRY_HOST' => 'example.org', + 'DBENTRY' => '/kunden/homepages/44/dexample/htdocs/example/www:d0000#CPU 6 #MEM 10240 #CGI 18 #NPROC 12 #TAID 46322755 #WERB 0 #LANG 2 #STAT 1', + 'SCRIPT_URL' => '/dummy.php/dummy/foo', + 'SCRIPT_URI' => 'http://www.example.org/dummy.php/dummy/foo', + 'HTTP_USER_AGENT' => 'Opera/9.80 (X11; Linux x86_64) Presto/2.12.388 Version/12.16', + 'HTTP_HOST' => 'www.example.org', + 'HTTP_ACCEPT' => 'text/html, application/xml;q=0.9, application/xhtml+xml, image/png, image/webp, image/jpeg, image/gif, image/x-xbitmap, */*;q=0.1', + 'HTTP_ACCEPT_LANGUAGE' => 'en,de-DE;q=0.9,de;q=0.8', + 'HTTP_ACCEPT_ENCODING' => 'gzip, deflate', + 'HTTP_COOKIE' => 'PHPSESSID=8c7853d7f639b3c6d24c224cf7d4cb1c', + 'HTTP_CONNECTION' => 'Keep-Alive', + 'HTTP_DNT' => '1', + 'PATH' => '/bin:/usr/bin', + 'SERVER_SIGNATURE' => '', + 'SERVER_SOFTWARE' => 'Apache', + 'SERVER_NAME' => 'example.org', + 'SERVER_ADDR' => '127.0.0.1', + 'SERVER_PORT' => '80', + 'REMOTE_ADDR' => '127.0.0.1', + 'DOCUMENT_ROOT' => '/kunden/homepages/44/dexample/htdocs/example/www', + 'SERVER_ADMIN' => 'webmaster@example.org', + 'SCRIPT_FILENAME' => '/kunden/homepages/44/dexample/htdocs/example/www/dummy.php', + 'REMOTE_PORT' => '35857', + 'REDIRECT_URL' => '/dummy.php/dummy/foo', + 'GATEWAY_INTERFACE' => 'CGI/1.1', + 'SERVER_PROTOCOL' => 'HTTP/1.1', + 'REQUEST_METHOD' => 'GET', + 'QUERY_STRING' => '', + 'REQUEST_URI' => '/dummy.php/dummy/foo', + 'SCRIPT_NAME' => '/dummy.php', + 'STATUS' => '200', + 'ORIG_PATH_INFO' => '/dummy/foo', + 'ORIG_PATH_TRANSLATED' => '/kunden/homepages/44/dexample/htdocs/example/www/dummy.php', + 'PHP_SELF' => '/dummy.php', + 'REQUEST_TIME_FLOAT' => 1377024137.8098, + 'REQUEST_TIME' => 1377024137, + 'argv' => array(), + 'argc' => 0, + ); + $this->assertEquals( + '/dummy/foo', SemanticScuttle_Environment::getServerPathInfo() + ); + } } ?>
\ No newline at end of file |