simplesamlphp / simplesamlphp-module-webauthn

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

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/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 +4 added lines, -4 removed lines

@@ -169,7 +169,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
         );
@@ -227,7 +227,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.");
@@ -251,7 +251,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.
 
@@ -269,7 +269,7 @@ 
             }
             // 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');
 

src/WebAuthn/WebAuthnRegistrationEvent.php

Spacing +4 added lines, -4 removed lines

@@ -380,8 +380,8 @@ 
          * §8.2 Step 2 Bullet 1: check signature
          */
         $retCode = openssl_verify($sigdata, $stmtDecoded['sig'], $keyResource, "sha256");
-        if ( $retCode !== 1) {
-            $this->fail("Packed signature mismatch (return code $retCode, for :authdata:".$attestationArray['authData']." - :clientDataHash:".$this->clientDataHash." - :signature:".$stmtDecoded['sig']."), attestation failed.");
+        if ($retCode !== 1) {
+            $this->fail("Packed signature mismatch (return code $retCode, for :authdata:" . $attestationArray['authData'] . " - :clientDataHash:" . $this->clientDataHash . " - :signature:" . $stmtDecoded['sig'] . "), attestation failed.");
         }
         $this->pass("x5c sig check passed.");
     }
@@ -482,7 +482,7 @@ 
     }
 
     // Keymaster 3 - KeyMint ???
-    private const ORIGINS_3 = [ // https://source.android.com/docs/security/features/keystore/tags#origin
+    private const ORIGINS_3 = [// https://source.android.com/docs/security/features/keystore/tags#origin
         0 => "GENERATED",
         1 => "DERIVED",
         2 => "IMPORTED",
@@ -700,7 +700,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!");