Skip to content
This repository was archived by the owner on Aug 7, 2023. It is now read-only.

Commit 4d4caf8

Browse files
Change attribute name
1 parent 99ff945 commit 4d4caf8

File tree

3 files changed

+11
-11
lines changed

3 files changed

+11
-11
lines changed

src/Authenticator/Authenticator.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ public function supports(Request $request): ?bool
4343

4444
if (null !== $payload && false !== is_string($payload[$mapping->userIdentifierClaim()] ?? null)) {
4545
$request->attributes->set(
46-
'_user_identifier_and_payload',
46+
'_user_identifier_claim_and_payload',
4747
[
4848
$mapping->userIdentifierClaim(),
4949
$payload,
@@ -59,8 +59,8 @@ public function supports(Request $request): ?bool
5959

6060
public function authenticate(Request $request): PassportInterface
6161
{
62-
[$userIdentifierClaim, $payload] = $request->attributes->get('_user_identifier_and_payload');
63-
$request->attributes->remove('_user_identifier_and_payload');
62+
[$userIdentifierClaim, $payload] = $request->attributes->get('_user_identifier_claim_and_payload');
63+
$request->attributes->remove('_user_identifier_claim_and_payload');
6464
$userBadge = new UserBadge($payload[$userIdentifierClaim], $this->makeUserLoader($payload));
6565
$passport = new SelfValidatingPassport($userBadge);
6666
$passport->setAttribute('_payload', $payload);

tests/Integration/AuthenticatorManagerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ public function testAuthenticatorControlledByManager()
3535

3636
$this->assertTrue($manager->supports($request));
3737
$this->assertTrue($request->attributes->has('_security_authenticators'));
38-
$this->assertTrue($request->attributes->has('_user_identifier_and_payload'));
38+
$this->assertTrue($request->attributes->has('_user_identifier_claim_and_payload'));
3939

4040
$controlledAuthenticators = $request->attributes->get('_security_authenticators');
4141

tests/Unit/Authenticator/AuthenticatorTest.php

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ public function testSupportValidRequest(Request $request)
3636
$authenticator = $this->createAuthenticator(new InMemoryUserProvider());
3737

3838
$this->assertTrue($authenticator->supports($request));
39-
$this->assertTrue($request->attributes->has('_user_identifier_and_payload'));
39+
$this->assertTrue($request->attributes->has('_user_identifier_claim_and_payload'));
4040
}
4141

4242
/**
@@ -47,7 +47,7 @@ public function testNotSupportInvalidRequest(Request $request)
4747
$authenticator = $this->createAuthenticator(new InMemoryUserProvider());
4848

4949
$this->assertFalse($authenticator->supports($request));
50-
$this->assertFalse($request->attributes->has('_user_identifier_and_payload'));
50+
$this->assertFalse($request->attributes->has('_user_identifier_claim_and_payload'));
5151
}
5252

5353
/**
@@ -57,11 +57,11 @@ public function testNotSupportInvalidRequest(Request $request)
5757
public function testAuthenticate(Request $request)
5858
{
5959
$authenticator = $this->createAuthenticatorWithInMemoryUserProvider(['valid' => []]);
60-
$authenticator->supports($request); // call to set `_user_identifier_and_payload` request attribute.
60+
$authenticator->supports($request); // call to set `_user_identifier_claim_and_payload` request attribute.
6161

6262
$passport = $authenticator->authenticate($request);
6363

64-
$this->assertFalse($request->attributes->has('_user_identifier_and_payload'));
64+
$this->assertFalse($request->attributes->has('_user_identifier_claim_and_payload'));
6565
$this->assertInstanceOf(SelfValidatingPassport::class, $passport);
6666
$this->assertTrue($passport->hasBadge(UserBadge::class));
6767
$this->assertIsArray($passport->getAttribute('_payload'));
@@ -80,9 +80,9 @@ public function testAuthenticateWithJWTPayloadAwareUserProvider(Request $request
8080
{
8181
$payload = [];
8282
$authenticator = $this->createAuthenticatorWithJWTPayloadAwareUserProvider($payload);
83-
$authenticator->supports($request); // call to set `_user_identifier_and_payload` request attribute.
83+
$authenticator->supports($request); // call to set `_user_identifier_claim_and_payload` request attribute.
8484
$payload = $request->attributes->get(
85-
'_user_identifier_and_payload'
85+
'_user_identifier_claim_and_payload'
8686
)[1]; // set for mock assert see: [[createAuthenticatorWithJWTPayloadAwareUserProvider()]]
8787
$passport = $authenticator->authenticate($request);
8888
$user = $passport->getBadge(UserBadge::class)->getUser();
@@ -98,7 +98,7 @@ public function testCreateAuthenticatedToken(Request $request)
9898
{
9999
$authenticator = $this->createAuthenticatorWithInMemoryUserProvider(['valid' => []]);
100100
$authenticator->supports($request);
101-
$payload = $request->attributes->get('_user_identifier_and_payload')[1];
101+
$payload = $request->attributes->get('_user_identifier_claim_and_payload')[1];
102102
$passport = $authenticator->authenticate($request);
103103
$token = $authenticator->createAuthenticatedToken($passport, 'test');
104104

0 commit comments

Comments
 (0)