simplesamlphp / simplesamlphp-module-webauthn

src/WebAuthn/WebAuthnAbstractEvent.php

Indentation +1 added lines, -1 removed lines

@@ -182,7 +182,7 @@
         return $this->credential;
     }
 
-   /**
+    /**
      * @return int
      */
     public function getAlgo(): int

src/Controller/RegProcess.php

Spacing +1 added lines, -1 removed lines

@@ -196,7 +196,7 @@
         $id = $this->authState::saveState($state, 'webauthn:request');
         if ($debugEnabled === true) {
             $response = new RunnableResponse(
-                function (WebAuthnRegistrationEvent $regObject, string $id) {
+                function(WebAuthnRegistrationEvent $regObject, string $id) {
                     echo $regObject->getDebugBuffer();
                     echo $regObject->getValidateBuffer();
                     echo "<form id='regform' method='POST' action='" .

src/Controller/WebAuthn.php

Indentation +1 added lines, -1 removed lines

@@ -109,7 +109,7 @@
             }
             return self::STATE_AUTH_ALLOWMGMT;
         } else { // in inflow, allow to check the management box; otherwise,
-                 // only auth
+                    // only auth
             $moduleConfig = Configuration::getOptionalConfig('module_webauthn.php')->toArray();
             return $moduleConfig['registration']['use_inflow_registration'] ? self::STATE_AUTH_ALLOWMGMT : self::STATE_AUTH_NOMGMT;
         }

Spacing +1 added lines, -1 removed lines

@@ -215,7 +215,7 @@
         $frontendData = [];
         $frontendData['challengeEncoded'] = $challengeEncoded;
         $frontendData['state'] = [];
-        foreach (['FIDO2Scope','FIDO2Username','FIDO2Displayname','requestTokenModel'] as $stateItem) {
+        foreach (['FIDO2Scope', 'FIDO2Username', 'FIDO2Displayname', 'requestTokenModel'] as $stateItem) {
             $frontendData['state'][$stateItem] = $state[$stateItem];
         }
 

src/WebAuthn/WebAuthnRegistrationEvent.php

Spacing +1 added lines, -1 removed lines

@@ -624,7 +624,7 @@
          * STEP 13 of the validation procedure in § 7.1 of the spec: is the algorithm the expected one?
          */
         if (in_array($arrayPK['3'], self::PK_ALGORITHM)) { // we requested -7 or -257, so want to see it here
-            $this->algo = (int)$arrayPK['3'];
+            $this->algo = (int) $arrayPK['3'];
             $this->pass("Public Key Algorithm is expected (" . implode(' or ', WebAuthnRegistrationEvent::PK_ALGORITHM) . ").");
         } else {
             $this->fail("Public Key Algorithm mismatch!");

src/Controller/PushbackUserPass.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
         // this is the confirmed username, we store it just like the Passwordless
         // one would have been
         
-        $state['Attributes'][$state['FIDO2AttributeStoringUsername']] = [ $request->request->get("username") ];
+        $state['Attributes'][$state['FIDO2AttributeStoringUsername']] = [$request->request->get("username")];
 
         // we deliberately do not store any additional attributes - these have
         // to be retrieved from the same authproc that would retrieve them

src/Controller/Supercharged.php

Spacing +1 added lines, -1 removed lines

@@ -215,7 +215,7 @@
         $frontendData = [];
         $frontendData['challengeEncoded'] = $challengeEncoded;
         $frontendData['state'] = [];
-        foreach (['FIDO2Scope','FIDO2Username','FIDO2Displayname','requestTokenModel'] as $stateItem) {
+        foreach (['FIDO2Scope', 'FIDO2Username', 'FIDO2Displayname', 'requestTokenModel'] as $stateItem) {
             $frontendData['state'][$stateItem] = $state[$stateItem];
         }
 

src/Controller/AuthProcess.php

Spacing +6 added lines, -6 removed lines

@@ -165,7 +165,7 @@ 
             base64_decode($request->request->get('client_data_raw')),
             $oneToken[0],
             $oneToken[1],
-            (int)$oneToken[4], // algo
+            (int) $oneToken[4], // algo
             base64_decode($request->request->get('signature')),
             $debugEnabled
         );
@@ -223,7 +223,7 @@ 
 
         if ($debugEnabled) {
             $response = new RunnableResponse(
-                function (WebAuthnAuthenticationEvent $authObject, array $state) {
+                function(WebAuthnAuthenticationEvent $authObject, array $state) {
                     echo $authObject->getDebugBuffer();
                     echo $authObject->getValidateBuffer();
                     echo "Debug mode, not continuing to " . ($state['FIDO2WantsRegister'] ? "credential registration page." : "destination.");
@@ -247,7 +247,7 @@ 
              * In passwordless, we're on our own. The one thing we know is the
              * username.
              */
-            $state['Attributes'][$state['FIDO2AttributeStoringUsername']] = [ $state['FIDO2Username'] ];
+            $state['Attributes'][$state['FIDO2AttributeStoringUsername']] = [$state['FIDO2Username']];
             // in case this authentication happened in the Supercharged context
             // it may be that there is an authprocfilter for WebAuthN, too.
             
@@ -260,19 +260,19 @@ 
             
             foreach ($state['IdPMetadata']['authproc'] as $index => $content) {
                 if ($content['class'] == "webauthn:WebAuthn") {
-                    unset( $state['IdPMetadata']['authproc'][$index] );
+                    unset($state['IdPMetadata']['authproc'][$index]);
                 }
             }
             // set an internal "authenticated passwordless" hint somewhere else
             // in $state, which the authproc can react upon
-            $state['Attributes']['internal:FIDO2PasswordlessAuthentication'] = [ $state['FIDO2Username'] ];
+            $state['Attributes']['internal:FIDO2PasswordlessAuthentication'] = [$state['FIDO2Username']];
             
             $this->authState::saveState($state, 'webauthn:request');
             
             // set a cookie to remember that the user has successfully used
             // Passwordless - on the Supercharged AuthSource, this can be used
             // to auto-trigger the FIDO2 authentication step next time
-            setcookie("SuccessfullyUsedPasswordlessBefore", "YES", time()+60*60*24*90, '/', "", true, true );
+            setcookie("SuccessfullyUsedPasswordlessBefore", "YES", time() + 60 * 60 * 24 * 90, '/', "", true, true);
             
             // now properly return our final state to the framework
             Source::completeAuth($state);