Skip to content

Commit 055a4f4

Browse files
committed
Track session usage when setting the token
1 parent 3787386 commit 055a4f4

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

Tests/Firewall/ContextListenerTest.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -411,9 +411,9 @@ protected function runSessionOnKernelResponse($newToken, $original = null)
411411

412412
private function handleEventWithPreviousSession($userProviders, UserInterface $user = null, RememberMeServicesInterface $rememberMeServices = null)
413413
{
414-
$user = $user ?: new User('foo', 'bar');
414+
$tokenUser = $user ?: new User('foo', 'bar');
415415
$session = new Session(new MockArraySessionStorage());
416-
$session->set('_security_context_key', serialize(new UsernamePasswordToken($user, '', 'context_key', ['ROLE_USER'])));
416+
$session->set('_security_context_key', serialize(new UsernamePasswordToken($tokenUser, '', 'context_key', ['ROLE_USER'])));
417417

418418
$request = new Request();
419419
$request->setSession($session);
@@ -442,6 +442,10 @@ private function handleEventWithPreviousSession($userProviders, UserInterface $u
442442
$listener(new RequestEvent($this->getMockBuilder(HttpKernelInterface::class)->getMock(), $request, HttpKernelInterface::MASTER_REQUEST));
443443

444444
if (null !== $usageIndex) {
445+
if (null !== $user) {
446+
++$usageIndex;
447+
}
448+
445449
$this->assertSame($usageIndex, $session->getUsageIndex());
446450
$tokenStorage->getToken();
447451
$this->assertSame(1 + $usageIndex, $session->getUsageIndex());

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
],
1818
"require": {
1919
"php": "^7.1.3",
20-
"symfony/security-core": "^4.4.7",
20+
"symfony/security-core": "^4.4.8",
2121
"symfony/http-foundation": "^3.4.40|^4.4.7|^5.0.7",
2222
"symfony/http-kernel": "^4.4",
2323
"symfony/property-access": "^3.4|^4.0|^5.0"

0 commit comments

Comments
 (0)