From e838aa9514bb0ade137b5a763534707be4801a89 Mon Sep 17 00:00:00 2001 From: Ferdinand Thiessen Date: Sun, 17 Mar 2024 20:20:43 +0100 Subject: [PATCH] fix(Logger): Warn on invalid `loglevel` configuration option Signed-off-by: Ferdinand Thiessen --- lib/private/Log.php | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/lib/private/Log.php b/lib/private/Log.php index ddccf87e07354..b8f6d26893024 100644 --- a/lib/private/Log.php +++ b/lib/private/Log.php @@ -282,7 +282,13 @@ public function getLogLevel($context): int { } $configLogLevel = $this->config->getValue('loglevel', ILogger::WARN); - return min(is_int($configLogLevel) ? $configLogLevel : ILogger::WARN, ILogger::FATAL); + if (is_numeric($configLogLevel)) { + return min((int)$configLogLevel, ILogger::FATAL); + } + + // Invalid configuration, warn the user and fall back to default level of WARN + error_log('Nextcloud configuration: "loglevel" is not a valid integer'); + return ILogger::WARN; } /**