Merge pull request #249 from cvergne/fix/assert-security-config

andrew-demb · web-flow · commit e65189cdd477 · 2025-06-06T12:51:54.000+02:00
[Fix] Avoid break on assert if security node is missing in config
diff --git a/src/DependencyInjection/GraphQLiteExtension.php b/src/DependencyInjection/GraphQLiteExtension.php
@@ -73,6 +73,9 @@ function($namespace): string {
             $namespaceType = [];
         }
 
+        if (!isset($config['security'])) {
+            $config['security'] = [];
+        }
         \assert(\is_array($config['security']));
         $enableLogin = $config['security']['enable_login'] ?? 'auto';
         $enableMe = $config['security']['enable_me'] ?? 'auto';

Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,9 @@ function($namespace): string {`
`73`	`73`	`$namespaceType = [];`
`74`	`74`	`}`
`75`	`75`
	`76`	`+ if (!isset($config['security'])) {`
	`77`	`+ $config['security'] = [];`
	`78`	`+ }`
`76`	`79`	`\assert(\is_array($config['security']));`
`77`	`80`	`$enableLogin = $config['security']['enable_login'] ?? 'auto';`
`78`	`81`	`$enableMe = $config['security']['enable_me'] ?? 'auto';`