Passed
Push — feature/package_updates ( ff6130 )
by Johan
03:15
created
Surfnet/StepupGateway/GatewayBundle/Service/ResponseRenderingService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -140,7 +140,7 @@
 block discarded – undo
140 140
 
141 141
         $httpResponse = (new Response)->setContent(
142 142
             $this->templateEngine->render(
143
-                '@default/gateway/'.$view.'.html.twig',
143
+                '@default/gateway/' . $view . '.html.twig',
144 144
                 $parameters
145 145
             )
146 146
         );
Please login to merge, or discard this patch.
GatewayBundle/Monolog/Formatter/GelfMessageToStringFormatter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
      * {@inheritdoc} the gelf message is an array which cannot be logged into a single line
42 42
      * By jsonencoding the message we can write it on a single line
43 43
      */
44
-    public function format(array|LogRecord $record)
44
+    public function format(array | LogRecord $record)
45 45
     {
46 46
         $message = $this->formatter->format($record);
47 47
 
Please login to merge, or discard this patch.
src/Surfnet/StepupGateway/GatewayBundle/Saml/ResponseBuilder.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -86,20 +86,20 @@
 block discarded – undo
86 86
     private function isValidResponseStatus(string $status): bool
87 87
     {
88 88
         return in_array($status, [
89
-            Constants::STATUS_SUCCESS,            // weeee!
90
-            Constants::STATUS_REQUESTER,          // Something is wrong with the AuthnRequest
91
-            Constants::STATUS_RESPONDER,          // Something went wrong with the Response
92
-            Constants::STATUS_VERSION_MISMATCH,   // The version of the request message was incorrect
89
+            Constants::STATUS_SUCCESS, // weeee!
90
+            Constants::STATUS_REQUESTER, // Something is wrong with the AuthnRequest
91
+            Constants::STATUS_RESPONDER, // Something went wrong with the Response
92
+            Constants::STATUS_VERSION_MISMATCH, // The version of the request message was incorrect
93 93
         ]);
94 94
     }
95 95
 
96 96
     private function isValidResponseSubStatus($subStatus): bool
97 97
     {
98 98
         return in_array($subStatus, [
99
-            Constants::STATUS_AUTHN_FAILED,               // failed authentication
99
+            Constants::STATUS_AUTHN_FAILED, // failed authentication
100 100
             Constants::STATUS_INVALID_ATTR,
101 101
             Constants::STATUS_INVALID_NAMEID_POLICY,
102
-            Constants::STATUS_NO_AUTHN_CONTEXT,           // insufficient Loa or Loa cannot be met
102
+            Constants::STATUS_NO_AUTHN_CONTEXT, // insufficient Loa or Loa cannot be met
103 103
             Constants::STATUS_NO_AVAILABLE_IDP,
104 104
             Constants::STATUS_NO_PASSIVE,
105 105
             Constants::STATUS_NO_SUPPORTED_IDP,
Please login to merge, or discard this patch.
src/Surfnet/StepupGateway/GatewayBundle/Controller/GatewayController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -236,7 +236,7 @@
 block discarded – undo
236 236
     public function render($view, array $parameters = [], ?Response $response = null): Response
237 237
     {
238 238
         return parent::render(
239
-            '@default/gateway/'.$view.'.html.twig',
239
+            '@default/gateway/' . $view . '.html.twig',
240 240
             $parameters,
241 241
             $response,
242 242
         );
Please login to merge, or discard this patch.
src/Surfnet/StepupGateway/GatewayBundle/Controller/ExceptionController.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@
 block discarded – undo
43 43
     /**
44 44
      * @return array View parameters 'title' and 'description'
45 45
      */
46
-    protected function getPageTitleAndDescription(\Exception|Throwable $exception): array
46
+    protected function getPageTitleAndDescription(\Exception | Throwable $exception): array
47 47
     {
48 48
         $translator = $this->getTranslator();
49 49
 
Please login to merge, or discard this patch.
Surfnet/StepupGateway/GatewayBundle/Controller/SecondFactorController.php 2 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -294,7 +294,7 @@
 block discarded – undo
294 294
             $form->addError(
295 295
                 new FormError(
296 296
                     $this->get('translator')
297
-                      ->trans('gateway.form.gateway_choose_second_factor.unknown_second_factor_type'),
297
+                        ->trans('gateway.form.gateway_choose_second_factor.unknown_second_factor_type'),
298 298
                 ),
299 299
             );
300 300
         }
Please login to merge, or discard this patch.
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
     public function selectSecondFactorForVerification(
81 81
         string $authenticationMode,
82 82
         Request $request,
83
-    ): Response|RedirectResponse {
83
+    ): Response | RedirectResponse {
84 84
         $this->supportsAuthenticationMode($authenticationMode);
85 85
         $context = $this->getResponseContext($authenticationMode);
86 86
         $originalRequestId = $context->getInResponseTo();
@@ -206,7 +206,7 @@  discard block
 block discarded – undo
206 206
     public function chooseSecondFactor(
207 207
         Request $request,
208 208
         string $authenticationMode,
209
-    ): Response|RedirectResponse|array {
209
+    ): Response | RedirectResponse | array {
210 210
         $this->supportsAuthenticationMode($authenticationMode);
211 211
         $context = $this->getResponseContext($authenticationMode);
212 212
         $originalRequestId = $context->getInResponseTo();
@@ -495,7 +495,7 @@  discard block
 block discarded – undo
495 495
     )]
496 496
     public function verifySmsSecondFactor(
497 497
         Request $request,
498
-    ): Response|RedirectResponse {
498
+    ): Response | RedirectResponse {
499 499
         $authenticationMode = $request->query->getString('authenticationMode', '');
500 500
         if ($authenticationMode === '') {
501 501
             $authenticationMode = $request->request->get('authenticationMode');
@@ -576,7 +576,7 @@  discard block
 block discarded – undo
576 576
     )]
577 577
     public function verifySmsSecondFactorChallenge(
578 578
         Request $request,
579
-    ): Response|array {
579
+    ): Response | array {
580 580
         $authenticationMode = $request->query->get('authenticationMode');
581 581
         if ($authenticationMode === '') {
582 582
             $authenticationMode = $request->request->get('authenticationMode');
Please login to merge, or discard this patch.
StepupGateway/SecondFactorOnlyBundle/Service/Gateway/ResponseValidator.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -69,7 +69,7 @@
 block discarded – undo
69 69
             if ($subjectNameIdFromResponse !== $secondFactor->getSecondFactorIdentifier()) {
70 70
                 throw new ReceivedInvalidSubjectNameIdException(
71 71
                     sprintf(
72
-                        'The nameID received from the GSSP (%s) did not match the selected second factor (%s). This '.
72
+                        'The nameID received from the GSSP (%s) did not match the selected second factor (%s). This ' .
73 73
                         'might be an indication someone is tampering with a GSSP. The authentication was started by %s',
74 74
                         $subjectNameIdFromResponse,
75 75
                         $secondFactor->getSecondFactorIdentifier(),
Please login to merge, or discard this patch.
StepupGateway/SecondFactorOnlyBundle/Service/Gateway/RespondService.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@
 block discarded – undo
102 102
         }
103 103
 
104 104
         $secondFactor = $this->secondFactorService->findByUuid($selectedSecondFactorUuid);
105
-        $loaLevel =  $this->secondFactorService->getLoaLevel($secondFactor);
105
+        $loaLevel = $this->secondFactorService->getLoaLevel($secondFactor);
106 106
         $this->responseValidator->validate($request, $secondFactor, $responseContext->getIdentityNameId());
107 107
 
108 108
         $authnContextClassRef = $this->loaAliasLookupService->findAliasByLoa($loaLevel);
Please login to merge, or discard this patch.
src/Surfnet/StepupGateway/GatewayBundle/Saml/Proxy/ProxyStateHandler.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -204,7 +204,7 @@  discard block
 block discarded – undo
204 204
 
205 205
     public function isSecondFactorFallback(): bool
206 206
     {
207
-        return (bool)$this->get('selected_second_factor_fallback');
207
+        return (bool) $this->get('selected_second_factor_fallback');
208 208
     }
209 209
 
210 210
     public function setGsspUserAttributes(string $subject, string $institution): ProxyStateHandler
@@ -217,12 +217,12 @@  discard block
 block discarded – undo
217 217
 
218 218
     public function getGsspUserAttributeSubject(): string
219 219
     {
220
-        return (string)$this->get('user_attribute_subject');
220
+        return (string) $this->get('user_attribute_subject');
221 221
     }
222 222
 
223 223
     public function getGsspUserAttributeInstitution(): string
224 224
     {
225
-        return (string)$this->get('user_attribute_institution');
225
+        return (string) $this->get('user_attribute_institution');
226 226
     }
227 227
 
228 228
     public function setVerifiedBySsoOn2faCookie(bool $isVerifiedByCookie): ProxyStateHandler
Please login to merge, or discard this patch.