diff options
| author | Lukas Reschke <lukas@statuscode.ch> | 2016-08-19 14:46:20 +0200 |
|---|---|---|
| committer | Lukas Reschke <lukas@statuscode.ch> | 2016-08-19 14:48:13 +0200 |
| commit | d50e7ee36c5607da848702ca36f7b34505c1622e (patch) | |
| tree | c2c4b9ac0783472417a97e777125b8b660495035 /tests/lib/AppFramework | |
| parent | cb7b44d33d276ccc0bc305d1af50c10eb5802191 (diff) | |
| download | nextcloud-server-d50e7ee36c5607da848702ca36f7b34505c1622e.tar.gz nextcloud-server-d50e7ee36c5607da848702ca36f7b34505c1622e.zip | |
Remove reading PATH_INFO from server variable
Having two code paths for this is unreliable and can lead to bugs. Also, in some cases Apache isn't setting the PATH_INFO variable when mod_rewrite is used.
Fixes https://github.com/nextcloud/server/issues/983
Diffstat (limited to 'tests/lib/AppFramework')
| -rw-r--r-- | tests/lib/AppFramework/Http/RequestTest.php | 16 |
1 files changed, 0 insertions, 16 deletions
diff --git a/tests/lib/AppFramework/Http/RequestTest.php b/tests/lib/AppFramework/Http/RequestTest.php index 8df81afeb3b..a3433e558d8 100644 --- a/tests/lib/AppFramework/Http/RequestTest.php +++ b/tests/lib/AppFramework/Http/RequestTest.php @@ -1063,22 +1063,6 @@ class RequestTest extends \Test\TestCase { $this->assertSame('www.owncloud.org', self::invokePrivate($request, 'getOverwriteHost')); } - public function testGetPathInfoWithSetEnv() { - $request = new Request( - [ - 'server' => [ - 'PATH_INFO' => 'apps/files/', - ] - ], - $this->secureRandom, - $this->config, - $this->csrfTokenManager, - $this->stream - ); - - $this->assertSame('apps/files/', $request->getPathInfo()); - } - /** * @expectedException \Exception * @expectedExceptionMessage The requested uri(/foo.php) cannot be processed by the script '/var/www/index.php') |