Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[SecurityBundle] Deprecate the enable_authenticator_manager option #47890

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 20, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions UPGRADE-6.2.md
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,11 @@ Security
* Change the signature of `TokenStorageInterface::setToken()` to `setToken(?TokenInterface $token)`
* Deprecate calling `TokenStorage::setToken()` or `UsageTrackingTokenStorage::setToken()` without arguments

SecurityBundle
--------------

* Deprecate the `security.enable_authenticator_manager` config option

Serializer
----------

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@ services:
- container.service_subscriber

security:
enable_authenticator_manager: true

providers:
main:
memory:
Expand Down
1 change: 1 addition & 0 deletions src/Symfony/Bundle/SecurityBundle/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ CHANGELOG
* Add `security.firewalls.logout.enable_csrf` to enable CSRF protection using the default CSRF token generator
* Add RFC6750 Access Token support to allow token-based authentication
* Add `security.firewalls.switch_user.target_route` option to configure redirect target route on switch user
* Deprecate the `security.enable_authenticator_manager` config option

6.1
---
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ public function getConfigTreeBuilder(): TreeBuilder
->end()
->booleanNode('hide_user_not_found')->defaultTrue()->end()
->booleanNode('erase_credentials')->defaultTrue()->end()
->booleanNode('enable_authenticator_manager')->defaultTrue()->end()
->booleanNode('enable_authenticator_manager')->setDeprecated('symfony/security-bundle', '6.2', 'The "%node%" option at "%path%" is deprecated.')->defaultTrue()->end()
->arrayNode('access_decision_manager')
->addDefaultsIfNotSet()
->children()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,6 @@ private function createContainer($sessionStorageOptions)

$config = [
'security' => [
'enable_authenticator_manager' => true,
'providers' => ['some_provider' => ['id' => 'foo']],
'firewalls' => ['some_firewall' => ['security' => false]],
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ protected function setUp(): void
public function testEventIsPropagated(string $configuredEvent, string $registeredEvent)
{
$this->container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => ['main' => ['pattern' => '/', 'http_basic' => true]],
]);

Expand Down Expand Up @@ -90,7 +89,6 @@ public function providePropagatedEvents(): array
public function testRegisterCustomListener()
{
$this->container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => ['main' => ['pattern' => '/', 'http_basic' => true]],
]);

Expand All @@ -111,7 +109,6 @@ public function testRegisterCustomListener()
public function testRegisterCustomSubscriber()
{
$this->container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => ['main' => ['pattern' => '/', 'http_basic' => true]],
]);

Expand All @@ -131,7 +128,6 @@ public function testRegisterCustomSubscriber()
public function testMultipleFirewalls()
{
$this->container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => ['main' => ['pattern' => '/', 'http_basic' => true], 'api' => ['pattern' => '/api', 'http_basic' => true]],
]);

Expand Down Expand Up @@ -161,7 +157,6 @@ public function testMultipleFirewalls()
public function testListenerAlreadySpecific()
{
$this->container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => ['main' => ['pattern' => '/', 'http_basic' => true]],
]);

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'access_decision_manager' => [
'allow_if_all_abstain' => true,
'allow_if_equal_granted_denied' => false,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => [
'memory' => [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'access_decision_manager' => [
'service' => 'app.access_decision_manager',
],
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'access_decision_manager' => [
'service' => 'app.access_decision_manager',
'strategy' => 'affirmative',
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'access_decision_manager' => [
'strategy_service' => 'app.custom_access_decision_strategy',
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
$this->load('container1.php');

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'password_hashers' => [
'JMS\FooBundle\Entity\User7' => [
'algorithm' => 'argon2i',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
use Symfony\Component\Security\Http\Authenticator\Passport\Badge\CsrfTokenBadge;

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => [
'main' => [
'required_badges' => [CsrfTokenBadge::class, 'RememberMeBadge'],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
$this->load('container1.php');

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'password_hashers' => [
'JMS\FooBundle\Entity\User7' => [
'algorithm' => 'bcrypt',
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'password_hashers' => [
'JMS\FooBundle\Entity\User1' => 'plaintext',
'JMS\FooBundle\Entity\User2' => [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => [
'memory' => $memory = [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => [
'memory' => [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => [
'memory' => [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => [
'memory' => [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => ['id' => 'foo'],
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
$this->load('merge_import.php');

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => ['id' => 'foo'],
],
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'firewalls' => [
'main' => [
'form_login' => [
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
$this->load('container1.php');

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'password_hashers' => [
'JMS\FooBundle\Entity\User7' => [
'algorithm' => 'argon2i',
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => [
'memory' => [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
<?php

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'providers' => [
'default' => ['id' => 'foo'],
],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
$this->load('container1.php');

$container->loadFromExtension('security', [
'enable_authenticator_manager' => true,
'password_hashers' => [
'JMS\FooBundle\Entity\User7' => [
'algorithm' => 'sodium',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<access-decision-manager allow-if-all-abstain="true" allow-if-equal-granted-denied="false" />

<provider name="default">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<provider name="default">
<memory>
<user identifier="foo" password="foo" roles="ROLE_USER" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<access-decision-manager service="app.access_decision_manager" />

<provider name="default">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<access-decision-manager service="app.access_decision_manager" strategy="affirmative" />

<provider name="default">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<access-decision-manager strategy-service="app.custom_access_decision_strategy" />

<provider name="default">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<import resource="container1.xml"/>
</imports>

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:password_hasher class="JMS\FooBundle\Entity\User7" algorithm="argon2i" memory-cost="256" time-cost="1" />
</sec:config>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<firewall name="main">
<required-badge>Symfony\Component\Security\Http\Authenticator\Passport\Badge\CsrfTokenBadge</required-badge>
<required-badge>RememberMeBadge</required-badge>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<import resource="container1.xml"/>
</imports>

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:password_hasher class="JMS\FooBundle\Entity\User7" algorithm="bcrypt" cost="15" />
</sec:config>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<password_hasher class="JMS\FooBundle\Entity\User1" algorithm="plaintext" />

<password_hasher class="JMS\FooBundle\Entity\User2" algorithm="sha1" encode-as-base64="false" iterations="5" />
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:providers>
<sec:provider name="with-dash" id="foo" />
</sec:providers>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:providers>
<sec:provider name="default" id="foo" />
</sec:providers>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:providers>
<sec:provider name="default" id="foo" />
</sec:providers>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:providers>
<sec:provider name="default" id="foo" />
</sec:providers>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<provider name="default" id="foo" />

<firewall name="main" provider="default">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<import resource="merge_import.xml"/>
</imports>

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:provider name="default" id="foo" />

<sec:firewall name="main" form-login="false">
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@
http://symfony.com/schema/dic/security
https://symfony.com/schema/dic/security/security-1.0.xsd">

<config enable-authenticator-manager="true">
<config>
<firewall name="main">
<form-login login-path="/login" />
</firewall>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<import resource="container1.xml"/>
</imports>

<sec:config enable-authenticator-manager="true">
<sec:config>
<sec:password_hasher class="JMS\FooBundle\Entity\User7" algorithm="argon2i" memory-cost="256" time-cost="1">
<sec:migrate-from>bcrypt</sec:migrate-from>
</sec:password_hasher>
Expand Down
Loading