Update & fix CS

BearerPipelineTest · Jul 21, 2022 · fd12513 · fd12513
1 parent cd2c0a0
commit fd12513
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/src/OAuth/ResourceOwner/AppleResourceOwner.php b/src/OAuth/ResourceOwner/AppleResourceOwner.php
@@ -146,7 +146,7 @@ protected function configureOptions(OptionsResolver $resolver)
 
     private static function jwtDecode(string $idToken)
     {
-        //// from http://stackoverflow.com/a/28748285/624544
+        // from http://stackoverflow.com/a/28748285/624544
         [, $jwt] = explode('.', $idToken, 3);
 
         // if the token was urlencoded, do some fixes to ensure that it is valid base64 encoded

diff --git a/src/OAuth/ResourceOwner/AzureResourceOwner.php b/src/OAuth/ResourceOwner/AzureResourceOwner.php
@@ -48,7 +48,7 @@ public function configure()
     public function getUserInformation(array $accessToken, array $extraParameters = [])
     {
         // from http://stackoverflow.com/a/28748285/624544
-        [, $jwt] = explode('.', (\array_key_exists('id_token', $accessToken) ? $accessToken['id_token'] : $accessToken['access_token']), 3);
+        [, $jwt] = explode('.', \array_key_exists('id_token', $accessToken) ? $accessToken['id_token'] : $accessToken['access_token'], 3);
 
         // if the token was urlencoded, do some fixes to ensure that it is valid base64 encoded
         $jwt = str_replace(['-', '_'], ['+', '/'], $jwt);

diff --git a/src/OAuth/ResourceOwner/KeycloakResourceOwner.php b/src/OAuth/ResourceOwner/KeycloakResourceOwner.php
@@ -57,9 +57,9 @@ protected function configureOptions(OptionsResolver $resolver)
 
         $normalizer = function (Options $options, $value) {
             return str_replace(
-              '{keycloak_url}',
-              $options['base_url'].'/realms/'.$options['realm'].'/protocol/'.$options['protocol'],
-              $value
+                '{keycloak_url}',
+                $options['base_url'].'/realms/'.$options['realm'].'/protocol/'.$options['protocol'],
+                $value
             );
         };
 

diff --git a/src/Security/OAuthUtils.php b/src/Security/OAuthUtils.php
@@ -170,7 +170,7 @@ public static function signRequest(
 
         // Remove query params from URL
         // Ref: Spec: 9.1.2
-        $url = sprintf('%s://%s%s%s', $url['scheme'], $url['host'], ($explicitPort ? ':'.$explicitPort : ''), $url['path'] ?? '');
+        $url = sprintf('%s://%s%s%s', $url['scheme'], $url['host'], $explicitPort ? ':'.$explicitPort : '', $url['path'] ?? '');
 
         // Parameters are sorted by name, using lexicographical byte value ordering.
         // Ref: Spec: 9.1.1 (1)

diff --git a/tests/Security/OAuthUtilsTest.php b/tests/Security/OAuthUtilsTest.php
@@ -315,7 +315,7 @@ private function getHttpUtils(string $generatedUrl = '/'): HttpUtils
         $urlGenerator
             ->expects($this->any())
             ->method('generate')
-            ->willReturn(($generatedUrl))
+            ->willReturn($generatedUrl)
         ;
 
         return new HttpUtils($urlGenerator);