Merge pull request #32 from bwaidelich/patch-1

BUGFIX: Prevent state cache from flooding the filesystem

Author: robertlemke

Classes/OAuthClient.php

@@ -359,6 +359,7 @@ public function finishAuthorization(string $stateIdentifier, string $code, strin
         if (empty($stateFromCache)) {
             throw new OAuthClientException(sprintf('OAuth: Finishing authorization failed because oAuth state %s could not be retrieved from the state cache.', $stateIdentifier), 1558956494);
         }
+        $this->stateCache->remove($stateIdentifier);
 
         $authorizationId = $stateFromCache['authorizationId'];
         $clientId = $stateFromCache['clientId'];
@@ -619,6 +620,7 @@ public function shutdownObject(): void
         try {
             if (random_int(1, 100 * $factor) <= ($this->garbageCollectionProbability * $factor)) {
                 $this->removeExpiredAuthorizations();
+                $this->stateCache->collectGarbage();
             }
         } catch (\Exception $e) {
         }