aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorChristian Weiske <cweiske@cweiske.de>2013-08-20 20:58:28 +0200
committerChristian Weiske <cweiske@cweiske.de>2013-08-20 20:58:28 +0200
commite6494f9767645940fa6af44b176d1b7c225bc4ed (patch)
tree0f1270c88819398440f34f5371d5f624bc4875be
parentf309a61b717a0ac3327d27406095a03d9226a617 (diff)
downloadsemanticscuttle-e6494f9767645940fa6af44b176d1b7c225bc4ed.tar.gz
semanticscuttle-e6494f9767645940fa6af44b176d1b7c225bc4ed.tar.bz2
Fix bug #161: URLs broken on 1&1 server
-rw-r--r--doc/ChangeLog5
-rw-r--r--src/SemanticScuttle/Environment.php26
-rw-r--r--src/SemanticScuttle/constants.php4
-rw-r--r--tests/SemanticScuttle/EnvironmentTest.php139
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