Skip to content

Commit 3d988ae

Browse files
committed
feat(settings): add big file upload setup checks
Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
1 parent bb2e8e0 commit 3d988ae

4 files changed

Lines changed: 71 additions & 0 deletions

File tree

apps/settings/composer/composer/autoload_classmap.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,7 @@
115115
'OCA\\Settings\\SetupChecks\\PhpDisabledFunctions' => $baseDir . '/../lib/SetupChecks/PhpDisabledFunctions.php',
116116
'OCA\\Settings\\SetupChecks\\PhpFreetypeSupport' => $baseDir . '/../lib/SetupChecks/PhpFreetypeSupport.php',
117117
'OCA\\Settings\\SetupChecks\\PhpGetEnv' => $baseDir . '/../lib/SetupChecks/PhpGetEnv.php',
118+
'OCA\\Settings\\SetupChecks\\PhpMaxFileSize' => $baseDir . '/../lib/SetupChecks/PhpMaxFileSize.php',
118119
'OCA\\Settings\\SetupChecks\\PhpMemoryLimit' => $baseDir . '/../lib/SetupChecks/PhpMemoryLimit.php',
119120
'OCA\\Settings\\SetupChecks\\PhpModules' => $baseDir . '/../lib/SetupChecks/PhpModules.php',
120121
'OCA\\Settings\\SetupChecks\\PhpOpcacheSetup' => $baseDir . '/../lib/SetupChecks/PhpOpcacheSetup.php',

apps/settings/composer/composer/autoload_static.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -130,6 +130,7 @@ class ComposerStaticInitSettings
130130
'OCA\\Settings\\SetupChecks\\PhpDisabledFunctions' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpDisabledFunctions.php',
131131
'OCA\\Settings\\SetupChecks\\PhpFreetypeSupport' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpFreetypeSupport.php',
132132
'OCA\\Settings\\SetupChecks\\PhpGetEnv' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpGetEnv.php',
133+
'OCA\\Settings\\SetupChecks\\PhpMaxFileSize' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpMaxFileSize.php',
133134
'OCA\\Settings\\SetupChecks\\PhpMemoryLimit' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpMemoryLimit.php',
134135
'OCA\\Settings\\SetupChecks\\PhpModules' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpModules.php',
135136
'OCA\\Settings\\SetupChecks\\PhpOpcacheSetup' => __DIR__ . '/..' . '/../lib/SetupChecks/PhpOpcacheSetup.php',

apps/settings/lib/AppInfo/Application.php

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,7 @@
5858
use OCA\Settings\SetupChecks\PhpDisabledFunctions;
5959
use OCA\Settings\SetupChecks\PhpFreetypeSupport;
6060
use OCA\Settings\SetupChecks\PhpGetEnv;
61+
use OCA\Settings\SetupChecks\PhpMaxFileSize;
6162
use OCA\Settings\SetupChecks\PhpMemoryLimit;
6263
use OCA\Settings\SetupChecks\PhpModules;
6364
use OCA\Settings\SetupChecks\PhpOpcacheSetup;
@@ -203,6 +204,7 @@ public function register(IRegistrationContext $context): void {
203204
$context->registerSetupCheck(PhpFreetypeSupport::class);
204205
$context->registerSetupCheck(PhpApcuConfig::class);
205206
$context->registerSetupCheck(PhpGetEnv::class);
207+
$context->registerSetupCheck(PhpMaxFileSize::class);
206208
$context->registerSetupCheck(PhpMemoryLimit::class);
207209
$context->registerSetupCheck(PhpModules::class);
208210
$context->registerSetupCheck(PhpOpcacheSetup::class);
Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
<?php
2+
3+
declare(strict_types=1);
4+
5+
/**
6+
* SPDX-FileCopyrightText: 2024 Nextcloud GmbH and Nextcloud contributors
7+
* SPDX-License-Identifier: AGPL-3.0-or-later
8+
*/
9+
10+
namespace OCA\Settings\SetupChecks;
11+
12+
use OCP\AppFramework\Services\IAppConfig;
13+
use OCP\IL10N;
14+
use OCP\IURLGenerator;
15+
use OCP\SetupCheck\ISetupCheck;
16+
use OCP\SetupCheck\SetupResult;
17+
use OCP\Util;
18+
19+
class PhpMaxFileSize implements ISetupCheck {
20+
public function __construct(
21+
private IL10N $l10n,
22+
private IAppConfig $appConfig,
23+
private IURLGenerator $urlGenerator,
24+
) {
25+
}
26+
27+
public function getCategory(): string {
28+
return 'php';
29+
}
30+
31+
public function getName(): string {
32+
return $this->l10n->t('PHP file size upload limit');
33+
}
34+
35+
public function run(): SetupResult {
36+
$upload_max_filesize = Util::computerFileSize(trim(ini_get('upload_max_filesize')));
37+
$post_max_size = Util::computerFileSize(trim(ini_get('post_max_size')));
38+
$max_input_time = (int)trim(ini_get('max_input_time'));
39+
$max_execution_time = (int)trim(ini_get('max_execution_time'));
40+
41+
$warnings = [];
42+
$recommendedSize = 16 * 1024 * 1024 * 1024;
43+
$recommendedTime = 3600;
44+
45+
// Check if the PHP upload limit is too low
46+
if ($upload_max_filesize < $recommendedSize || $post_max_size < $recommendedSize) {
47+
$warnings[] = $this->l10n->t('The PHP upload_max_filesize and/or post_max_size is too low. A file size of at least %1$s is recommended. Current value: %2$s.', [
48+
Util::humanFileSize($recommendedSize),
49+
Util::humanFileSize(min($upload_max_filesize, $post_max_size)),
50+
]);
51+
}
52+
53+
// Check if the PHP execution time is too low
54+
if (($max_input_time < $recommendedTime || $max_execution_time < $recommendedTime) && $max_input_time !== -1 && $max_execution_time !== -1) {
55+
$warnings[] = $this->l10n->t('The PHP max_input_time and/or max_execution_time is too low. A time of at least %1$s is recommended. Current value: %2$s.', [
56+
$recommendedTime,
57+
min($max_input_time, $max_execution_time),
58+
]);
59+
}
60+
61+
if (!empty($warnings)) {
62+
return SetupResult::warning(join(' ', $warnings), $this->urlGenerator->linkToDocs('admin-big-file-upload'));
63+
}
64+
65+
return SetupResult::success();
66+
}
67+
}

0 commit comments

Comments
 (0)