mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #984 from nextcloud/dont-read-server-variable 

Remove reading PATH_INFO from server variable
This commit is contained in:
Roeland Jago Douma 2016-08-20 18:21:31 +02:00 committed by GitHub
parent 8007ea60fb d50e7ee36c
commit 8fbec6aae3
2 changed files with 0 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/private/AppFramework/Http/Request.php
View File

@ -700,10 +700,6 @@ class Request implements \ArrayAccess, \Countable, IRequest {
* @return string|false Path info or false when not found * @return string|false Path info or false when not found
*/ */
public function getPathInfo() { public function getPathInfo() {
if(isset($this->server['PATH_INFO'])) {
return $this->server['PATH_INFO'];
}
$pathInfo = $this->getRawPathInfo(); $pathInfo = $this->getRawPathInfo();
// following is taken from \Sabre\HTTP\URLUtil::decodePathSegment // following is taken from \Sabre\HTTP\URLUtil::decodePathSegment
$pathInfo = rawurldecode($pathInfo); $pathInfo = rawurldecode($pathInfo);

16
tests/lib/AppFramework/Http/RequestTest.php
View File

@ -1063,22 +1063,6 @@ class RequestTest extends \Test\TestCase {
$this->assertSame('www.owncloud.org', self::invokePrivate($request, 'getOverwriteHost')); $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 * @expectedException \Exception
* @expectedExceptionMessage The requested uri(/foo.php) cannot be processed by the script '/var/www/index.php') * @expectedExceptionMessage The requested uri(/foo.php) cannot be processed by the script '/var/www/index.php')