diff options
| author | marco44 <cousinmarc@gmail.com> | 2018-05-16 12:59:58 +0200 |
|---|---|---|
| committer | marco44 <cousinmarc@gmail.com> | 2018-05-17 14:28:32 +0200 |
| commit | 32fd0919c1fc546bf84b5ce2da54dd59c57f4085 (patch) | |
| tree | 4ee5eb05c4f4438d4b7e3893996b464e103f9b48 /lib | |
| parent | 0011bfb64bedbfa412ffd2190433fe0b51830ce9 (diff) | |
| download | nextcloud-server-32fd0919c1fc546bf84b5ce2da54dd59c57f4085.tar.gz nextcloud-server-32fd0919c1fc546bf84b5ce2da54dd59c57f4085.zip | |
Make LargeFileHelper.php faster by avoiding execs as much as possible
Signed-off-by: Marc Cousin <cousinmarc@gmail.com>
Diffstat (limited to 'lib')
| -rwxr-xr-x[-rw-r--r--] | lib/private/LargeFileHelper.php | 20 |
1 files changed, 14 insertions, 6 deletions
diff --git a/lib/private/LargeFileHelper.php b/lib/private/LargeFileHelper.php index ea848f83622..d6dcbeedb48 100644..100755 --- a/lib/private/LargeFileHelper.php +++ b/lib/private/LargeFileHelper.php @@ -117,7 +117,7 @@ class LargeFileHelper { public function getFileSizeViaCurl($fileName) { if (\OC::$server->getIniWrapper()->getString('open_basedir') === '') { $encodedFileName = rawurlencode($fileName); - $ch = curl_init("file://$encodedFileName"); + $ch = curl_init("file:///$encodedFileName"); curl_setopt($ch, CURLOPT_NOBODY, true); curl_setopt($ch, CURLOPT_RETURNTRANSFER, true); curl_setopt($ch, CURLOPT_HEADER, true); @@ -185,14 +185,22 @@ class LargeFileHelper { * @return int */ public function getFileMtime($fullPath) { - if (\OC_Helper::is_function_enabled('exec')) { - $os = strtolower(php_uname('s')); - if (strpos($os, 'linux') !== false) { - return $this->exec('stat -c %Y ' . escapeshellarg($fullPath)); + try { + $result = filemtime($fullPath); + } catch (\Exception $e) { + $result =- 1; + } + if ($result < 0) { + if (\OC_Helper::is_function_enabled('exec')) { + $os = strtolower(php_uname('s')); + if (strpos($os, 'linux') !== false) { + return $this->exec('stat -c %Y ' . escapeshellarg($fullPath)); + } } } + return $result; + - return filemtime($fullPath); } protected function exec($cmd) { |