OpenConext / Stepup-Gateway

src/Surfnet/StepupGateway/GatewayBundle/Saml/ResponseBuilder.php

Spacing +6 added lines, -6 removed lines

@@ -86,20 +86,20 @@
     private function isValidResponseStatus(string $status): bool
     {
         return in_array($status, [
-            Constants::STATUS_SUCCESS,            // weeee!
-            Constants::STATUS_REQUESTER,          // Something is wrong with the AuthnRequest
-            Constants::STATUS_RESPONDER,          // Something went wrong with the Response
-            Constants::STATUS_VERSION_MISMATCH,   // The version of the request message was incorrect
+            Constants::STATUS_SUCCESS, // weeee!
+            Constants::STATUS_REQUESTER, // Something is wrong with the AuthnRequest
+            Constants::STATUS_RESPONDER, // Something went wrong with the Response
+            Constants::STATUS_VERSION_MISMATCH, // The version of the request message was incorrect
         ]);
     }
 
     private function isValidResponseSubStatus($subStatus): bool
     {
         return in_array($subStatus, [
-            Constants::STATUS_AUTHN_FAILED,               // failed authentication
+            Constants::STATUS_AUTHN_FAILED, // failed authentication
             Constants::STATUS_INVALID_ATTR,
             Constants::STATUS_INVALID_NAMEID_POLICY,
-            Constants::STATUS_NO_AUTHN_CONTEXT,           // insufficient Loa or Loa cannot be met
+            Constants::STATUS_NO_AUTHN_CONTEXT, // insufficient Loa or Loa cannot be met
             Constants::STATUS_NO_AVAILABLE_IDP,
             Constants::STATUS_NO_PASSIVE,
             Constants::STATUS_NO_SUPPORTED_IDP,

src/Surfnet/StepupGateway/GatewayBundle/Controller/GatewayController.php

Spacing +1 added lines, -1 removed lines

@@ -236,7 +236,7 @@
     public function render($view, array $parameters = [], ?Response $response = null): Response
     {
         return parent::render(
-            '@default/gateway/'.$view.'.html.twig',
+            '@default/gateway/' . $view . '.html.twig',
             $parameters,
             $response,
         );

src/Surfnet/StepupGateway/GatewayBundle/Controller/ExceptionController.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     /**
      * @return array View parameters 'title' and 'description'
      */
-    protected function getPageTitleAndDescription(\Exception|Throwable $exception): array
+    protected function getPageTitleAndDescription(\Exception | Throwable $exception): array
     {
         $translator = $this->getTranslator();
 

src/Surfnet/StepupGateway/GatewayBundle/Controller/SecondFactorController.php

Spacing +4 added lines, -4 removed lines

@@ -78,7 +78,7 @@ 
     public function selectSecondFactorForVerification(
         string $authenticationMode,
         Request $request,
-    ): Response|RedirectResponse {
+    ): Response | RedirectResponse {
         $this->supportsAuthenticationMode($authenticationMode);
         $context = $this->getResponseContext($authenticationMode);
         $originalRequestId = $context->getInResponseTo();
@@ -192,7 +192,7 @@ 
     public function chooseSecondFactor(
         Request $request,
         string $authenticationMode,
-    ): Response|RedirectResponse|array {
+    ): Response | RedirectResponse | array {
         $this->supportsAuthenticationMode($authenticationMode);
         $context = $this->getResponseContext($authenticationMode);
         $originalRequestId = $context->getInResponseTo();
@@ -479,7 +479,7 @@ 
     )]
     public function verifySmsSecondFactor(
         Request $request,
-    ): Response|RedirectResponse {
+    ): Response | RedirectResponse {
         if (!$request->get('authenticationMode', false)) {
             throw new RuntimeException('Unable to determine the authentication mode in the SMS verification action');
         }
@@ -559,7 +559,7 @@ 
     )]
     public function verifySmsSecondFactorChallenge(
         Request $request,
-    ): Response|array {
+    ): Response | array {
         if (!$request->get('authenticationMode', false)) {
             throw new RuntimeException('Unable to determine the authentication mode in the SMS challenge action');
         }

src/Surfnet/StepupGateway/SecondFactorOnlyBundle/Service/Gateway/ResponseValidator.php

Spacing +1 added lines, -1 removed lines

@@ -69,7 +69,7 @@
             if ($subjectNameIdFromResponse !== $secondFactor->getSecondFactorIdentifier()) {
                 throw new ReceivedInvalidSubjectNameIdException(
                     sprintf(
-                        'The nameID received from the GSSP (%s) did not match the selected second factor (%s). This '.
+                        'The nameID received from the GSSP (%s) did not match the selected second factor (%s). This ' .
                         'might be an indication someone is tampering with a GSSP. The authentication was started by %s',
                         $subjectNameIdFromResponse,
                         $secondFactor->getSecondFactorIdentifier(),

src/Surfnet/StepupGateway/SecondFactorOnlyBundle/Service/Gateway/RespondService.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
         }
 
         $secondFactor = $this->secondFactorService->findByUuid($selectedSecondFactorUuid);
-        $loaLevel =  $this->secondFactorService->getLoaLevel($secondFactor);
+        $loaLevel = $this->secondFactorService->getLoaLevel($secondFactor);
         $this->responseValidator->validate($request, $secondFactor, $responseContext->getIdentityNameId());
 
         $authnContextClassRef = $this->loaAliasLookupService->findAliasByLoa($loaLevel);

src/Surfnet/StepupGateway/GatewayBundle/Saml/Proxy/ProxyStateHandler.php

Spacing +3 added lines, -3 removed lines

@@ -204,7 +204,7 @@ 
 
     public function isSecondFactorFallback(): bool
     {
-        return (bool)$this->get('selected_second_factor_fallback');
+        return (bool) $this->get('selected_second_factor_fallback');
     }
 
     public function setGsspUserAttributes(string $subject, string $institution): ProxyStateHandler
@@ -217,12 +217,12 @@ 
 
     public function getGsspUserAttributeSubject(): string
     {
-        return (string)$this->get('user_attribute_subject');
+        return (string) $this->get('user_attribute_subject');
     }
 
     public function getGsspUserAttributeInstitution(): string
     {
-        return (string)$this->get('user_attribute_institution');
+        return (string) $this->get('user_attribute_institution');
     }
 
     public function setVerifiedBySsoOn2faCookie(bool $isVerifiedByCookie): ProxyStateHandler

src/Surfnet/StepupGateway/SecondFactorOnlyBundle/Service/Gateway/GsspFallbackService.php

Spacing +1 added lines, -1 removed lines

@@ -162,7 +162,7 @@
             $this->stateHandler->getGsspUserAttributeSubject(),
             $this->stateHandler->getGsspUserAttributeInstitution(),
             $this->config->getGssp(),
-            (string)$this->stateHandler->getPreferredLocale()
+            (string) $this->stateHandler->getPreferredLocale()
         );
     }
 }

src/Surfnet/StepupGateway/GatewayBundle/Monolog/Logger/AuthenticationLogger.php

Spacing +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
             'institution'           => $secondFactor->getInstitution(),
             'authentication_result' => $context->isSecondFactorVerified() ? 'OK' : 'FAILED',
             'resulting_loa'         => (string) $loa,
-            'sso' => $context->isVerifiedBySsoOn2faCookie() ? 'YES': 'NO',
+            'sso' => $context->isVerifiedBySsoOn2faCookie() ? 'YES' : 'NO',
         ];
 
         if ($context->isVerifiedBySsoOn2faCookie()) {