Plugin: OAuth2: Fix catch BadMethodCallException when refreshing toke…

…n - refs BT#19734
chamilo · Jul 22, 2022 · 9388a5d · 9388a5d
1 parent 65bfb3e
commit 9388a5d
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/plugin/oauth2/index.php b/plugin/oauth2/index.php
@@ -8,6 +8,10 @@
  */
 
 /** @var OAuth2 $oAuth2Plugin */
+
+use League\OAuth2\Client\Provider\Exception\IdentityProviderException;
+use League\OAuth2\Client\Token\AccessToken;
+
 $oAuth2Plugin = OAuth2::create();
 
 if ($oAuth2Plugin->get(OAuth2::SETTING_ENABLE) === 'true') {
@@ -30,19 +34,16 @@
     }
 
     if (ChamiloSession::has('oauth2AccessToken')) {
-        $accessToken = new \League\OAuth2\Client\Token\AccessToken(ChamiloSession::read('oauth2AccessToken'));
+        $accessToken = new AccessToken(ChamiloSession::read('oauth2AccessToken'));
         if ($accessToken->hasExpired()) {
             $provider = $oAuth2Plugin->getProvider();
             try {
-                try {
-                    $newAccessToken = $provider->getAccessToken('refresh_token', [
-                        'refresh_token' => $accessToken->getRefreshToken(),
-                    ]);
-                } catch (BadMethodCallException $exception) {
-                    online_logout(null, true);
-                }
+                $newAccessToken = $provider->getAccessToken(
+                    'refresh_token',
+                    ['refresh_token' => $accessToken->getRefreshToken()]
+                );
                 ChamiloSession::write('oauth2AccessToken', $newAccessToken->jsonSerialize());
-            } catch (\League\OAuth2\Client\Provider\Exception\IdentityProviderException $exception) {
+            } catch (IdentityProviderException|\BadMethodCallException $e) {
                 online_logout(null, true);
             }
         }