OpenConext / Stepup-Gateway

src/Surfnet/StepupGateway/GatewayBundle/Service/ResponseRenderingService.php

Spacing +1 added lines, -1 removed lines

@@ -140,7 +140,7 @@
 
         $httpResponse = (new Response)->setContent(
             $this->templateEngine->render(
-                '@default/gateway/'.$view.'.html.twig',
+                '@default/gateway/' . $view . '.html.twig',
                 $parameters
             )
         );

src/Surfnet/StepupGateway/GatewayBundle/Monolog/Formatter/GelfMessageToStringFormatter.php

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
      * {@inheritdoc} the gelf message is an array which cannot be logged into a single line
      * By jsonencoding the message we can write it on a single line
      */
-    public function format(array|LogRecord $record)
+    public function format(array | LogRecord $record)
     {
         $message = $this->formatter->format($record);
 

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/GatewayBundle/Monolog/Logger/AuthenticationLogger.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
         ResponseContext     $sfoResponseContext,
         ResponseContext     $ssoResponseContext,
     ) {
-        $this->secondFactorService  = $secondFactorService;
+        $this->secondFactorService = $secondFactorService;
         $this->authenticationChannelLogger = $authenticationChannelLogger;
         $this->sfoResponseContext = $sfoResponseContext;
         $this->ssoResponseContext = $ssoResponseContext;
@@ -69,7 +69,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()) {

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