bug #60419 [SecurityBundle] normalize string values to a single ExposeSecurityLevel instance (xabbuh)

chalasr · chalasr · commit e67b3621d732 · 2025-05-14T13:16:35.000+02:00
This PR was merged into the 7.3 branch. Discussion ---------- [SecurityBundle] normalize string values to a single ExposeSecurityLevel instance | Q | A | ------------- | --- | Branch? | 7.3 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | | License | MIT Commits ------- d120484 normalize string values to a single ExposeSecurityLevel instance
diff --git a/src/Symfony/Bundle/SecurityBundle/DependencyInjection/MainConfiguration.php b/src/Symfony/Bundle/SecurityBundle/DependencyInjection/MainConfiguration.php
@@ -76,7 +76,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                     ->setDeprecated('symfony/security-bundle', '7.3', 'The "%node%" option is deprecated and will be removed in 8.0. Use the "expose_security_errors" option instead.')
                 ->end()
                 ->enumNode('expose_security_errors')
-                    ->beforeNormalization()->ifString()->then(fn ($v) => ['value' => ExposeSecurityLevel::tryFrom($v)])->end()
+                    ->beforeNormalization()->ifString()->then(fn ($v) => ExposeSecurityLevel::tryFrom($v))->end()
                     ->values(ExposeSecurityLevel::cases())
                     ->defaultValue(ExposeSecurityLevel::None)
                 ->end()
diff --git a/src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/MainConfigurationTest.php b/src/Symfony/Bundle/SecurityBundle/Tests/DependencyInjection/MainConfigurationTest.php
@@ -254,6 +254,9 @@ public static function provideHideUserNotFoundData(): iterable
         yield [['expose_security_errors' => ExposeSecurityLevel::None], ExposeSecurityLevel::None];
         yield [['expose_security_errors' => ExposeSecurityLevel::AccountStatus], ExposeSecurityLevel::AccountStatus];
         yield [['expose_security_errors' => ExposeSecurityLevel::All], ExposeSecurityLevel::All];
+        yield [['expose_security_errors' => 'none'], ExposeSecurityLevel::None];
+        yield [['expose_security_errors' => 'account_status'], ExposeSecurityLevel::AccountStatus];
+        yield [['expose_security_errors' => 'all'], ExposeSecurityLevel::All];
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -254,6 +254,9 @@ public static function provideHideUserNotFoundData(): iterable`
`254`	`254`	`yield [['expose_security_errors' => ExposeSecurityLevel::None], ExposeSecurityLevel::None];`
`255`	`255`	`yield [['expose_security_errors' => ExposeSecurityLevel::AccountStatus], ExposeSecurityLevel::AccountStatus];`
`256`	`256`	`yield [['expose_security_errors' => ExposeSecurityLevel::All], ExposeSecurityLevel::All];`
	`257`	`+ yield [['expose_security_errors' => 'none'], ExposeSecurityLevel::None];`
	`258`	`+ yield [['expose_security_errors' => 'account_status'], ExposeSecurityLevel::AccountStatus];`
	`259`	`+ yield [['expose_security_errors' => 'all'], ExposeSecurityLevel::All];`
`257`	`260`	`}`
`258`	`261`
`259`	`262`	`/**`