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/EndpointMessages/Authorization/AuthorizationRequest.php

Spacing +2 added lines, -2 removed lines

@@ -97,11 +97,11 @@
      * @throws \Exception
      */
     public function validate() {
-        if(!$this->responseType) {
+        if (!$this->responseType) {
             throw new \Exception('Missing response_type parameter');
         }
 
-        if(!$this->clientId) {
+        if (!$this->clientId) {
             throw new \Exception('Missing client_id parameter');
         }
         return true;

src/OAuth2/OpenID/ResponseTypes/CodeResponseType.php

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/Credentials/IDToken.php

Spacing +2 added lines, -2 removed lines

@@ -19,12 +19,12 @@
     public function __construct(array $claims)
     {
         $missingClaims = array_diff(self::REQUIRED_CLAIMS, array_keys($claims));
-        if(!empty($missingClaims)) {
+        if (!empty($missingClaims)) {
             throw new \Exception('Missing claims : '.implode(', ', $missingClaims));
         }
 
         $undefinedClaims = array_diff(array_keys($claims), self::DEFINED_CLAIMS);
-        if(!empty($undefinedClaims)) {
+        if (!empty($undefinedClaims)) {
             throw new \Exception('Undefined claims : '.implode(', ', $undefinedClaims));
         }
 

src/OAuth2/OpenID/Endpoints/AuthorizationEndpoint.php

Spacing +2 added lines, -2 removed lines

@@ -67,13 +67,13 @@
         foreach (explode(' ', $data['response_type']) as $responseTypeName) {
             $responseType = $this->server->getResponseTypeRepository()->getResponseType($responseTypeName);
             if (!$responseType) {
-                throw new OAuthException('invalid_request', 'Unknown response_type : ' . $responseTypeName);
+                throw new OAuthException('invalid_request', 'Unknown response_type : '.$responseTypeName);
             }
             $responseType->verifyRequest($request);
             $responseTypes[] = $responseType;
         }
 
-        if(empty($responseTypes)) {
+        if (empty($responseTypes)) {
             throw new OAuthException('invalid_request', 'Invalid response_type parameter');
         }
         else if (count($responseTypes) == 1) {

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');
         }
 

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];