Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit23f1226

Browse files
committed
bug#37283 [SecurityBundle] Fix CookieClearingLogoutListener DI configuration (wouterj)
This PR was merged into the 5.1 branch.Discussion----------[SecurityBundle] Fix CookieClearingLogoutListener DI configuration| Q | A| ------------- | ---| Branch? | 5.1| Bug fix? | yes| New feature? | no| Deprecations? | no| Tickets |Fix#37282| License | MIT| Doc PR | -Commits-------4786988 Fix CookieClearingLogoutListener DI configuration
2 parents8bb0897 +4786988 commit23f1226

File tree

6 files changed

+45
-2
lines changed

6 files changed

+45
-2
lines changed

‎src/Symfony/Bundle/SecurityBundle/Resources/config/security_listeners.xml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@
5454

5555
<serviceid="security.logout.listener.session"class="Symfony\Component\Security\Http\EventListener\SessionLogoutListener"abstract="true" />
5656

57-
<serviceid="security.logout.listener.cookie_clearing"class="Symfony\Component\Security\Http\Logout\CookieClearingLogoutHandler"abstract="true" />
57+
<serviceid="security.logout.listener.cookie_clearing"class="Symfony\Component\Security\Http\EventListener\CookieClearingLogoutListener"abstract="true" />
5858

5959
<serviceid="security.logout.listener.default"class="Symfony\Component\Security\Http\EventListener\DefaultLogoutListener"abstract="true">
6060
<argumenttype="service"id="security.http_utils" />

‎src/Symfony/Bundle/SecurityBundle/Tests/Functional/LogoutTest.php‎

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,8 @@
1111

1212
namespaceSymfony\Bundle\SecurityBundle\Tests\Functional;
1313

14+
useSymfony\Component\BrowserKit\Cookie;
15+
1416
class LogoutTestextends AbstractWebTestCase
1517
{
1618
/**
@@ -62,11 +64,25 @@ public function testCsrfTokensAreClearedOnLogout(array $options)
6264
*/
6365
publicfunctiontestAccessControlDoesNotApplyOnLogout(array$options)
6466
{
65-
$client =$this->createClient($options + ['test_case' =>'LogoutAccess','root_config' =>'config.yml']);
67+
$client =$this->createClient($options + ['test_case' =>'Logout','root_config' =>'config_access.yml']);
68+
69+
$client->request('POST','/login', ['_username' =>'johannes','_password' =>'test']);
70+
$client->request('GET','/logout');
71+
72+
$this->assertRedirect($client->getResponse(),'/');
73+
}
74+
75+
publicfunctiontestCookieClearingOnLogout()
76+
{
77+
$client =$this->createClient(['test_case' =>'Logout','root_config' =>'config_cookie_clearing.yml']);
78+
79+
$cookieJar =$client->getCookieJar();
80+
$cookieJar->set(newCookie('flavor','chocolate',strtotime('+1 day'),null,'somedomain'));
6681

6782
$client->request('POST','/login', ['_username' =>'johannes','_password' =>'test']);
6883
$client->request('GET','/logout');
6984

7085
$this->assertRedirect($client->getResponse(),'/');
86+
$this->assertNull($cookieJar->get('flavor'));
7187
}
7288
}
Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,27 @@
1+
imports:
2+
-{ resource: ./../config/framework.yml }
3+
4+
security:
5+
encoders:
6+
Symfony\Component\Security\Core\User\User:plaintext
7+
8+
providers:
9+
in_memory:
10+
memory:
11+
users:
12+
johannes:{ password: test, roles: [ROLE_USER] }
13+
14+
firewalls:
15+
default:
16+
form_login:
17+
check_path:login
18+
remember_me:true
19+
require_previous_session:false
20+
logout:
21+
delete_cookies:
22+
flavor:{ path: null, domain: somedomain }
23+
stateless:true
24+
25+
access_control:
26+
-{ path: ^/login$, roles: IS_AUTHENTICATED_ANONYMOUSLY }
27+
-{ path: .*, roles: IS_AUTHENTICATED_FULLY }

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp