OpenConext / Stepup-Gateway

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

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
         $response = $this->responseProxy->createProxyResponse(
             $responseContext->reconstituteAssertion(),
             $responseContext->getDestination(),
-            (string)$grantedLoa
+            (string) $grantedLoa
         );
 
         $logger->notice(sprintf(

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

Spacing +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
     private function addAuthenticationStatementTo(Assertion $newAssertion, Assertion $assertion)
     {
         $newAssertion->setAuthnInstant($assertion->getAuthnInstant());
-        $newAssertion->setAuthnContextClassRef((string)$this->intrinsicLoa);
+        $newAssertion->setAuthnContextClassRef((string) $this->intrinsicLoa);
 
         $authority = $assertion->getAuthenticatingAuthority();
         $newAssertion->setAuthenticatingAuthority(

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

Indentation +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function inResponseToMatches($inResponseTo)
     {
-         return $this->getInResponseTo() === $inResponseTo;
+            return $this->getInResponseTo() === $inResponseTo;
     }
 
     /**

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

Spacing +6 added lines, -6 removed lines

@@ -95,20 +95,20 @@
     private function isValidResponseStatus($status)
     {
         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)
     {
         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/Saml/ResponseContext.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
         $this->samlEntityService      = $samlEntityService;
         $this->stateHandler           = $stateHandler;
         $this->logger                 = $logger;
-        $this->generationTime         = is_null($now) ? new DateTime('now', new DateTimeZone('UTC')): $now;
+        $this->generationTime         = is_null($now) ? new DateTime('now', new DateTimeZone('UTC')) : $now;
     }
 
     /**

src/Surfnet/StepupGateway/GatewayBundle/Form/Type/SendSmsChallengeType.php

Spacing +1 added lines, -1 removed lines

@@ -29,7 +29,7 @@
     {
         $builder->add('send_challenge', SubmitType::class, [
             'label' => 'gateway.form.gateway_send_sms_challenge.button.send_challenge',
-            'attr' => [ 'class' => 'btn btn-primary' ],
+            'attr' => ['class' => 'btn btn-primary'],
         ]);
     }
 

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

Indentation +1 added lines, -1 removed lines

@@ -271,7 +271,7 @@
             $form->addError(
                 new FormError(
                     $this->get('translator')
-                      ->trans('gateway.form.gateway_choose_second_factor.unknown_second_factor_type')
+                        ->trans('gateway.form.gateway_choose_second_factor.unknown_second_factor_type')
                 )
             );
         }

Spacing +2 added lines, -2 removed lines

@@ -247,7 +247,7 @@ 
 
             // Filter the selected second factor from the array collection
             $secondFactorFiltered = $secondFactors->filter(
-                function ($secondFactor) use ($secondFactorType) {
+                function($secondFactor) use ($secondFactorType) {
                     return $secondFactorType === $secondFactor->secondFactorType;
                 }
             );
@@ -392,7 +392,7 @@ 
         $form = $this->createForm(VerifyYubikeyOtpType::class, $command)->handleRequest($request);
         $cancelForm = $this->buildCancelAuthenticationForm($authenticationMode)->handleRequest($request);
 
-        if ($form->isSubmitted()  && $form->isValid()) {
+        if ($form->isSubmitted() && $form->isValid()) {
             $result = $this->getStepupService()->verifyYubikeyOtp($command);
             if ($result->didOtpVerificationFail()) {
                 $form->addError(

src/Kernel.php

Spacing +11 added lines, -11 removed lines

@@ -32,7 +32,7 @@ 
 
     public function registerBundles(): iterable
     {
-        $contents = require $this->getProjectDir().'/config/bundles.php';
+        $contents = require $this->getProjectDir() . '/config/bundles.php';
         foreach ($contents as $class => $envs) {
             if ($envs[$this->environment] ?? $envs['all'] ?? false) {
                 yield new $class();
@@ -47,23 +47,23 @@ 
 
     protected function configureContainer(ContainerBuilder $container, LoaderInterface $loader): void
     {
-        $container->addResource(new FileResource($this->getProjectDir().'/config/bundles.php'));
+        $container->addResource(new FileResource($this->getProjectDir() . '/config/bundles.php'));
         $container->setParameter('container.dumper.inline_class_loader', \PHP_VERSION_ID < 70400 || $this->debug);
         $container->setParameter('container.dumper.inline_factories', true);
-        $confDir = $this->getProjectDir().'/config';
+        $confDir = $this->getProjectDir() . '/config';
 
-        $loader->load($confDir.'/{packages}/*'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/{packages}/'.$this->environment.'/*'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/{services}'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/{services}_'.$this->environment.self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{packages}/*' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{packages}/' . $this->environment . '/*' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{services}' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{services}_' . $this->environment . self::CONFIG_EXTS, 'glob');
     }
 
     protected function configureRoutes(RouteCollectionBuilder $routes): void
     {
-        $confDir = $this->getProjectDir().'/config';
+        $confDir = $this->getProjectDir() . '/config';
 
-        $routes->import($confDir.'/{routes}/'.$this->environment.'/*'.self::CONFIG_EXTS, '/', 'glob');
-        $routes->import($confDir.'/{routes}/*'.self::CONFIG_EXTS, '/', 'glob');
-        $routes->import($confDir.'/{routes}'.self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/{routes}/' . $this->environment . '/*' . self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/{routes}/*' . self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/{routes}' . self::CONFIG_EXTS, '/', 'glob');
     }
 }

tests/features/bootstrap/ServiceProviderContext.php

Spacing +5 added lines, -5 removed lines

@@ -177,7 +177,7 @@ 
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
 
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**
@@ -213,7 +213,7 @@ 
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
 
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**
@@ -241,7 +241,7 @@ 
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
 
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**
@@ -282,7 +282,7 @@ 
         );
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**
@@ -309,7 +309,7 @@ 
         );
         $request = Saml2AuthnRequest::createNew($authnRequest);
         $query = $request->buildRequestQuery();
-        $this->getSession()->visit($request->getDestination().'?'.$query);
+        $this->getSession()->visit($request->getDestination() . '?' . $query);
     }
 
     /**