Check if backend is null before passing it to strtolower

BastianLedererIcinga · BastianLedererIcinga · commit 53102a4d5b6f · 2026-01-08T10:58:19.000+01:00
Passing null to non nullable parameters of built-in functions is deprecated since PHP-8.1
diff --git a/library/Icinga/Authentication/User/UserBackend.php b/library/Icinga/Authentication/User/UserBackend.php
@@ -181,7 +181,7 @@ public static function create($name, ?ConfigObject $backendConfig = null)
             $name = $backendConfig->name;
         }
 
-        if (! ($backendType = strtolower($backendConfig->backend))) {
+        if ($backendConfig->backend === null || ! ($backendType = strtolower($backendConfig->backend))) {
             throw new ConfigurationError(
                 'Authentication configuration for user backend "%s" is missing the \'backend\' directive',
                 $name

Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ public static function create($name, ?ConfigObject $backendConfig = null)`
`181`	`181`	`$name = $backendConfig->name;`
`182`	`182`	`}`
`183`	`183`
`184`		`- if (! ($backendType = strtolower($backendConfig->backend))) {`
	`184`	`+ if ($backendConfig->backend === null \|\| ! ($backendType = strtolower($backendConfig->backend))) {`
`185`	`185`	`throw new ConfigurationError(`
`186`	`186`	`'Authentication configuration for user backend "%s" is missing the \'backend\' directive',`
`187`	`187`	`$name`