Merge pull request #10 from mercuryseries/patch-2

Add TreeBuilder return type declaration and remove cannotBeEmpty for domain
dneustadt · May 31, 2023 · b43623f · b43623f
2 parents b38013e + 755a295
commit b43623f
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -10,7 +10,7 @@
 
 class Configuration implements ConfigurationInterface
 {
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('dneustadt_csrf_cookie');
         $rootNode = $treeBuilder->getRootNode();
@@ -22,7 +22,7 @@ public function getConfigTreeBuilder()
             ->scalarNode('name')->cannotBeEmpty()->defaultValue('XSRF-TOKEN')->end()
             ->integerNode('expire')->defaultValue(0)->end()
             ->scalarNode('path')->cannotBeEmpty()->defaultValue('/')->end()
-            ->scalarNode('domain')->cannotBeEmpty()->defaultValue(null)->end()
+            ->scalarNode('domain')->defaultNull()->end()
             ->booleanNode('secure')->defaultFalse()->end()
             ->scalarNode('header')->cannotBeEmpty()->defaultValue('X-XSRF-TOKEN')->end()
             ->scalarNode('sameSite')->cannotBeEmpty()->defaultValue(Cookie::SAMESITE_LAX)->end()