php-guard / oauth-server

src/OAuth2/EndpointMessages/Authorization/ErrorResponse.php

Spacing +5 added lines, -5 removed lines

@@ -37,18 +37,18 @@ 
                                 ?string $errorUri = null, ?string $state = null)
     {
         if ($this->containsNotAsciiChar($error)) {
-            throw new \Exception('Malformed error type. Expect ascii string. Got : ' . $error);
+            throw new \Exception('Malformed error type. Expect ascii string. Got : '.$error);
         }
 
         if (!in_array($error, self::errors)) {
-            throw new \Exception('Unexpected error type. Expect one of : ' . implode(', ', self::errors));
+            throw new \Exception('Unexpected error type. Expect one of : '.implode(', ', self::errors));
         }
 
         $redirectUri = Uri::withQueryValue($redirectUri, 'error', $error);
 
         if ($errorDescription) {
             if ($this->containsNotAsciiChar($errorDescription)) {
-                throw new \Exception('Malformed error description. Expect ascii string. Got : ' . $errorDescription);
+                throw new \Exception('Malformed error description. Expect ascii string. Got : '.$errorDescription);
             }
 
             $redirectUri = Uri::withQueryValue($redirectUri, 'error_description', $errorDescription);
@@ -56,13 +56,13 @@ 
 
         if ($errorUri) {
             if ($this->containsNotAsciiChar($errorUri)) {
-                throw new \Exception('Malformed error uri. Expect ascii string. Got : ' . $errorUri);
+                throw new \Exception('Malformed error uri. Expect ascii string. Got : '.$errorUri);
             }
 
             $redirectUri = Uri::withQueryValue($redirectUri, 'error_uri', $errorUri);
         }
 
-        if($state) {
+        if ($state) {
             $redirectUri = Uri::withQueryValue($redirectUri, 'state', $state);
         }
 

src/OAuth2/ResponseTypes/ResponseTypeInterface.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      * @return array
      */
     public function handle(ServerRequestInterface $request, ResourceOwnerInterface $resourceOwner,
-                           RegisteredClient $client, ?array $scope = null): array;
+                            RegisteredClient $client, ?array $scope = null): array;
 
     public function getDefaultResponseMode(): string;
 

src/OAuth2/OpenID/ResponseTypes/CodeResponseType.php

Indentation +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
      * @return array
      */
     public function handle(ServerRequestInterface $request, ResourceOwnerInterface $resourceOwner,
-                           RegisteredClient $client, ?array $scope = null): array
+                            RegisteredClient $client, ?array $scope = null): array
     {
         // TODO: Implement handle() method.
     }

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
             );
         }
 
-        if(!$state && $this->configurationRepository->getConfig(Config::ENFORCE_STATE)) {
+        if (!$state && $this->configurationRepository->getConfig(Config::ENFORCE_STATE)) {
             throw new OAuthException('invalid_request',
                 'Missing a required parameter : state',
                 'http://openid.net/specs/openid-connect-core-1_0.html#AuthorizationEndpoint'

src/OAuth2/OpenID/ResponseTypes/IdTokenResponseType.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
      * @return array
      */
     public function handle(ServerRequestInterface $request, ResourceOwnerInterface $resourceOwner,
-                           RegisteredClient $client, ?array $scope = null): array
+                            RegisteredClient $client, ?array $scope = null): array
     {
 
     }

src/OAuth2/Endpoints/AuthorizationEndpoint.php

Spacing +1 added lines, -1 removed lines

@@ -221,7 +221,7 @@
             if (!$scopePolicyManager->checkScope($client, $scope)) {
                 $supportedScopes = implode(', ', $scopePolicyManager->getSupportedScopes($client));
                 throw new OAuthException('invalid_scope',
-                    'Some of requested scopes are not supported. Scope supported : ' . $supportedScopes,
+                    'Some of requested scopes are not supported. Scope supported : '.$supportedScopes,
                     'https://tools.ietf.org/html/rfc6749#section-4.1');
             }
 

src/OAuth2/Endpoints/TokenEndpoint.php

Spacing +6 added lines, -6 removed lines

@@ -36,7 +36,7 @@ 
         $grantType = $this->server->getGrantTypeRepository()->getGrantType($grantTypeName);
         if (!$grantType) {
             return new ErrorResponse('unsupported_grant_type',
-                'Unsupported grant type : ' . $grantTypeName,
+                'Unsupported grant type : '.$grantTypeName,
                 'https://tools.ietf.org/html/rfc6749#section-5.2');
         }
 
@@ -54,18 +54,18 @@ 
         } catch (OAuthException $e) {
             if ($e->getError() == 'invalid_client' && $request->hasHeader('Authorization')) {
                 return new ErrorResponse($e->getError(),
-                    'Client authentication failed : ' . $e->getMessage(),
+                    'Client authentication failed : '.$e->getMessage(),
                     $e->getErrorUri(), 401, [
                         'WWW-Authenticate' => 'Basic'
                     ]);
             } else {
                 return new ErrorResponse($e->getError(),
-                    'Client authentication failed : ' . $e->getMessage(),
+                    'Client authentication failed : '.$e->getMessage(),
                     $e->getErrorUri(), 401);
             }
         }
 
-        if(!$client) {
+        if (!$client) {
             if (!isset($request->getParsedBody()['client_id'])) {
                 return new ErrorResponse('invalid_request',
                     'Client authentication not included, missing a parameter : client_id : ',
@@ -81,7 +81,7 @@ 
 
             if ($client->hasCredentials()) {
                 return new ErrorResponse('invalid_client',
-                    'Client authentication failed : ' . $guard->getError(),
+                    'Client authentication failed : '.$guard->getError(),
                     'https://tools.ietf.org/html/rfc6749#section-5.2', 401, [
                         'WWW-Authenticate' => 'Basic'
                     ]);
@@ -90,7 +90,7 @@ 
 
         if (is_array($client->getSupportedGrantTypes()) && !in_array($grantType->getUri(), $client->getSupportedGrantTypes())) {
             return new ErrorResponse('unauthorized_client',
-                'Unauthorized grant type : ' . $grantType->getUri(),
+                'Unauthorized grant type : '.$grantType->getUri(),
                 'https://tools.ietf.org/html/rfc6749#section-5.2');
         }
 

Braces +1 added lines, -2 removed lines

@@ -96,8 +96,7 @@
 
         try {
             return $grantType->grant($request, $client);
-        }
-        catch (OAuthException $e) {
+        } catch (OAuthException $e) {
             return new ErrorResponse($e->getError(),
                 $e->getErrorDescription(),
                 $e->getErrorUri());

src/OAuth2/Repositories/ClientAuthenticatorRepository.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
      */
     public function __construct(array $authenticators)
     {
-        if(!isset($authenticators[ClientPasswordAuthenticator::class])) {
+        if (!isset($authenticators[ClientPasswordAuthenticator::class])) {
             throw new \Exception('Authorization server MUST support the HTTP Basic authentication scheme');
         }
         $this->authenticators = $authenticators;

src/OAuth2/Repositories/StorageRepository.php

Spacing +3 added lines, -3 removed lines

@@ -34,10 +34,10 @@ 
     public function __construct(array $storages)
     {
         foreach (self::STORAGES_INTERFACES as $name => $interface) {
-            if(!isset($storages[$name])) {
+            if (!isset($storages[$name])) {
                 throw new \Exception("Missing storage '$name'");
             }
-            if(!is_a($storages[$name], $interface)) {
+            if (!is_a($storages[$name], $interface)) {
                 throw new \Exception("Storage '$name' must implement '$interface'");
             }
             $this->storages[$name] = $storages[$name];
@@ -50,7 +50,7 @@ 
      * @throws \Exception
      */
     public function getStorage(string $name) {
-        if(!isset($this->storages[$name])) {
+        if (!isset($this->storages[$name])) {
             throw new \Exception("Unknown storage '$name'");
         }
         return $this->storages[$name];

src/OAuth2/Repositories/ConfigurationRepository.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
     public function __construct(array $config = [])
     {
         if (array_diff(self::MINIMAL_CONFIG, array_keys($config))) {
-            throw new \Exception('Missing minimal configuration. Required : ' . implode(', ', self::MINIMAL_CONFIG));
+            throw new \Exception('Missing minimal configuration. Required : '.implode(', ', self::MINIMAL_CONFIG));
         }
 
         $this->config = array_merge(self::DEFAULT_CONFIG, $config);
@@ -52,7 +52,7 @@ 
      * @throws \Exception
      */
     public function getConfig(string $name) {
-        if(!array_key_exists($name, $this->config)) {
+        if (!array_key_exists($name, $this->config)) {
             throw new \Exception("Unknown config '$name' in ".implode(', ', array_keys($this->config)));
         }
         return $this->config[$name];