Merge pull request #33644 from nextcloud/backport/32216/stable24

[stable24] Respect user settings in php.ini if they are big enough

1 files changed, 13 insertions, 6 deletions

diff --git a/lib/base.php b/lib/base.php
index 66e57459a91..b601e4b19f2 100644
--- a/lib/base.php
+++ b/lib/base.php
@@ -623,16 +623,23 @@ class OC {
 			throw new \RuntimeException('Could not set timezone to UTC');
 		}
 
+
 		//try to configure php to enable big file uploads.
-		//this doesn´t work always depending on the web server and php configuration.
-		//Let´s try to overwrite some defaults anyway
+		//this doesn´t work always depending on the webserver and php configuration.
+		//Let´s try to overwrite some defaults if they are smaller than 1 hour
+
+		if (intval(@ini_get('max_execution_time') ?? 0) < 3600) {
+			@ini_set('max_execution_time', strval(3600));
+		}
+
+		if (intval(@ini_get('max_input_time') ?? 0) < 3600) {
+			@ini_set('max_input_time', strval(3600));
+		}
 
-		//try to set the maximum execution time to 60min
+		//try to set the maximum execution time to the largest time limit we have
 		if (strpos(@ini_get('disable_functions'), 'set_time_limit') === false) {
-			@set_time_limit(3600);
+			@set_time_limit(max(intval(@ini_get('max_execution_time')), intval(@ini_get('max_input_time'))));
 		}
-		@ini_set('max_execution_time', '3600');
-		@ini_set('max_input_time', '3600');
 
 		self::setRequiredIniValues();
 		self::handleAuthHeaders();