Allow 0 again.

Author: dereuromark

config/app.example.php

@@ -9,7 +9,7 @@
 
 return [
 	'Queue' => [
-		// seconds of running time after which the worker process will terminate (0 = unlimited)
+		// seconds of running time after which the worker process will terminate (0 = unlimited, but not recommended)
 		'workerLifetime' => 60, // 1 minutes
 		// Legacy: 'workermaxruntime' is deprecated but still supported
 

docs/sections/configuration.md

@@ -39,7 +39,7 @@ You may create a file called `app_queue.php` inside your `config` folder (NOT th
     // Legacy: 'workermaxruntime' is deprecated but still supported
     ```
 
-  **Important:** This value cannot be 0. A value of 0 would mean "unlimited" which is not supported and will throw a RuntimeException. This setting is required to prevent workers from running indefinitely and potentially piling up if spawned faster than they can terminate, which could overload your server.
+  **Important:** While 0 (unlimited) is technically allowed, it is **strongly discouraged**. Using 0 means workers will run indefinitely, which can lead to workers piling up if spawned faster than they can naturally terminate (e.g., when there are no jobs), potentially overloading your server.
 
   If you need workers to run for extended periods, use a very large value (e.g., 86400 for 24 hours). However, it's recommended to use shorter durations (e.g., 60-300 seconds) with cronjob respawning for better control and safety.
 

src/Queue/Config.php

@@ -37,7 +37,8 @@ public static function defaultworkertimeout(): int {
 	}
 
 	/**
-	 * Seconds of running time after which the worker will terminate (0 = unlimited)
+	 * Seconds of running time after which the worker will terminate.
+	 * Note: 0 = unlimited is allowed but not recommended. Use a non-zero value for better control.
 	 *
 	 * @return int
 	 */

src/Queue/Processor.php

@@ -175,12 +175,12 @@ public function run(array $args): int {
 			}
 
 			$workerLifetime = Configure::read('Queue.workerLifetime') ?? Configure::read('Queue.workermaxruntime');
-			if (!$workerLifetime) {
+			if ($workerLifetime === null) {
 				throw new RuntimeException('Queue.workerLifetime (or deprecated workermaxruntime) config is required');
 			}
-			$maxRuntime = $config['maxruntime'] ?: $workerLifetime;
+			$maxRuntime = $config['maxruntime'] ?: (int)$workerLifetime;
 			// check if we are over the maximum runtime and end processing if so.
-			if ((time() - $startTime) >= $maxRuntime) {
+			if ($maxRuntime > 0 && (time() - $startTime) >= $maxRuntime) {
 				$this->exit = true;
 				$this->io->out('Reached runtime of ' . (time() - $startTime) . ' Seconds (Max ' . $maxRuntime . '), terminating.');
 			}