Set revokeRefreshTokens prop to true by default, added method to chan…

…ge, cleaned up constructor.
thephpleague · Jun 1, 2021 · c808d13 · c808d13
1 parent 5cf9d07
commit c808d13
Showing 1 changed file with 12 additions and 5 deletions.
diff --git a/src/AuthorizationServer.php b/src/AuthorizationServer.php
@@ -82,7 +82,7 @@ class AuthorizationServer implements EmitterAwareInterface
     /**
      * @var bool
      */
-    private $revokeRefreshTokens;
+    private $revokeRefreshTokens = true;
 
     /**
      * New server instance.
@@ -93,16 +93,14 @@ class AuthorizationServer implements EmitterAwareInterface
      * @param CryptKey|string                $privateKey
      * @param string|Key                     $encryptionKey
      * @param null|ResponseTypeInterface     $responseType
-     * @param bool                           $revokeRefreshTokens
      */
     public function __construct(
         ClientRepositoryInterface $clientRepository,
         AccessTokenRepositoryInterface $accessTokenRepository,
         ScopeRepositoryInterface $scopeRepository,
         $privateKey,
         $encryptionKey,
-        ResponseTypeInterface $responseType = null,
-        bool $revokeRefreshTokens = true
+        ResponseTypeInterface $responseType = null
     ) {
         $this->clientRepository = $clientRepository;
         $this->accessTokenRepository = $accessTokenRepository;
@@ -122,7 +120,6 @@ public function __construct(
         }
 
         $this->responseType = $responseType;
-        $this->revokeRefreshTokens = $revokeRefreshTokens;
     }
 
     /**
@@ -242,4 +239,14 @@ public function setDefaultScope($defaultScope)
     {
         $this->defaultScope = $defaultScope;
     }
+
+    /**
+     * Sets wether to revoke refresh tokens or not (for all grant types).
+     *
+     * @param bool $revokeRefreshTokens
+     */
+    public function setRevokeRefreshTokens(bool $revokeRefreshTokens): void
+    {
+        $this->revokeRefreshTokens = $revokeRefreshTokens;
+    }
 }