Skip to content

Commit b61b984

Browse files
author
Jordy Kiesebrink
committed
fix(120): Credential Revocation - (Pg)SQL error : Datatype mismatch
1 parent ff14fe0 commit b61b984

File tree

1 file changed

+12
-6
lines changed

1 file changed

+12
-6
lines changed

src/Service/CredentialsRevoker/DoctrineCredentialsRevoker.php

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,16 +40,17 @@ public function revokeCredentialsForUser(UserInterface $user): void
4040

4141
$this->entityManager->createQueryBuilder()
4242
->update(AccessToken::class, 'at')
43-
->set('at.revoked', true)
43+
->set('at.revoked', ':revoked')
4444
->where('at.userIdentifier = :userIdentifier')
45+
->setParameter('revoked', true)
4546
->setParameter('userIdentifier', $userIdentifier)
4647
->getQuery()
4748
->execute();
4849

4950
$queryBuilder = $this->entityManager->createQueryBuilder();
5051
$queryBuilder
5152
->update(RefreshToken::class, 'rt')
52-
->set('rt.revoked', true)
53+
->set('rt.revoked', ':revoked')
5354
->where($queryBuilder->expr()->in(
5455
'rt.accessToken',
5556
$this->entityManager->createQueryBuilder()
@@ -58,14 +59,16 @@ public function revokeCredentialsForUser(UserInterface $user): void
5859
->where('at.userIdentifier = :userIdentifier')
5960
->getDQL()
6061
))
62+
->setParameter('revoked', true)
6163
->setParameter('userIdentifier', $userIdentifier)
6264
->getQuery()
6365
->execute();
6466

6567
$this->entityManager->createQueryBuilder()
6668
->update(AuthorizationCode::class, 'ac')
67-
->set('ac.revoked', true)
69+
->set('ac.revoked', ':revoked')
6870
->where('ac.userIdentifier = :userIdentifier')
71+
->setParameter('revoked', true)
6972
->setParameter('userIdentifier', $userIdentifier)
7073
->getQuery()
7174
->execute();
@@ -78,15 +81,16 @@ public function revokeCredentialsForClient(AbstractClient $client): void
7881

7982
$this->entityManager->createQueryBuilder()
8083
->update(AccessToken::class, 'at')
81-
->set('at.revoked', true)
84+
->set('at.revoked', ':revoked')
8285
->where('at.client = :client')
8386
->setParameter('client', $doctrineClient->getIdentifier(), 'string')
87+
->setParameter('revoked', true)
8488
->getQuery()
8589
->execute();
8690

8791
$queryBuilder = $this->entityManager->createQueryBuilder();
8892
$queryBuilder->update(RefreshToken::class, 'rt')
89-
->set('rt.revoked', true)
93+
->set('rt.revoked', ':revoked')
9094
->where($queryBuilder->expr()->in(
9195
'rt.accessToken',
9296
$this->entityManager->createQueryBuilder()
@@ -96,14 +100,16 @@ public function revokeCredentialsForClient(AbstractClient $client): void
96100
->getDQL()
97101
))
98102
->setParameter('client', $doctrineClient->getIdentifier(), 'string')
103+
->setParameter('revoked', true)
99104
->getQuery()
100105
->execute();
101106

102107
$this->entityManager->createQueryBuilder()
103108
->update(AuthorizationCode::class, 'ac')
104-
->set('ac.revoked', true)
109+
->set('ac.revoked', ':revoked')
105110
->where('ac.client = :client')
106111
->setParameter('client', $doctrineClient->getIdentifier(), 'string')
112+
->setParameter('revoked', true)
107113
->getQuery()
108114
->execute();
109115
}

0 commit comments

Comments
 (0)