simplesamlphp / simplesamlphp-module-webauthn

lib/Store.php

Spacing +1 added lines, -1 removed lines

@@ -120,7 +120,7 @@
         }
 
         if (!is_array($config)) {
-            throw new Exception('Invalid configuration for consent store option: '.var_export($config, true));
+            throw new Exception('Invalid configuration for consent store option: ' . var_export($config, true));
         }
 
         if (!array_key_exists(0, $config)) {

www/managetoken.php

Switch Indentation +13 added lines, -13 removed lines

@@ -26,18 +26,18 @@
     throw new Exception("Attempt to access the token management page unauthenticated.");
 }
 switch ($_POST['submit']) {
-    case "NEVERMIND":
-        Auth\ProcessingChain::resumeProcessing($state);
-        break;
-    case "DELETE":
-        if ($state['FIDO2AuthSuccessful'] == $_POST['credId']) {
-            throw new Exception("Attempt to delete the currently used credential despite UI preventing this.");
-        }
-        $store = $state['webauthn:store'];
-        $store->deleteTokenData($_POST['credId']);
-        Auth\ProcessingChain::resumeProcessing($state);
-        break;
-    default:
-        throw new Exception("Unknown submit button state.");
+        case "NEVERMIND":
+            Auth\ProcessingChain::resumeProcessing($state);
+            break;
+        case "DELETE":
+            if ($state['FIDO2AuthSuccessful'] == $_POST['credId']) {
+                throw new Exception("Attempt to delete the currently used credential despite UI preventing this.");
+            }
+            $store = $state['webauthn:store'];
+            $store->deleteTokenData($_POST['credId']);
+            Auth\ProcessingChain::resumeProcessing($state);
+            break;
+        default:
+            throw new Exception("Unknown submit button state.");
 }
 

www/webauthn.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
 use SimpleSAML\Logger;
 use SimpleSAML\Module;
 use SimpleSAML\Utils;
-use SimpleSAML\XHTML\Template;;
+use SimpleSAML\XHTML\Template; ;
 
 $globalConfig = Configuration::getInstance();
 

www/authprocess.php

Spacing +2 added lines, -2 removed lines

@@ -82,10 +82,10 @@
 if ($debugEnabled) {
     echo $authObject->debugBuffer;
     echo $authObject->validateBuffer;
-    echo "Debug mode, not continuing to ". ($state['FIDO2WantsRegister'] ? "credential registration page." : "destination.");
+    echo "Debug mode, not continuing to " . ($state['FIDO2WantsRegister'] ? "credential registration page." : "destination.");
 } else {
     if ($state['FIDO2WantsRegister']) {
-        header("Location: ".Module::getModuleURL('webauthn/webauthn.php?StateId='.urlencode($id)));
+        header("Location: " . Module::getModuleURL('webauthn/webauthn.php?StateId=' . urlencode($id)));
     } else {
         Auth\ProcessingChain::resumeProcessing($state);
     }

lib/WebAuthn/WebAuthnRegistrationEvent.php

Spacing +7 added lines, -7 removed lines

@@ -174,16 +174,16 @@ 
              */
             $certProps = openssl_x509_parse($this->der2pem($stmtDecoded['x5c'][0]));
             $this->debugBuffer .= "Attestation Certificate:" . print_r($certProps, true) . "<br/>";
-            if ($certProps['version'] != 2 ||                                                                      /** §8.2.1 Bullet 1 */
-                    $certProps['subject']['OU'] != "Authenticator Attestation" ||                                  /** §8.2.1 Bullet 2 [Subject-OU] */
-                    !isset($certProps['subject']['CN']) ||                                                         /** §8.2.1 Bullet 2 [Subject-CN] */
+            if ($certProps['version'] != 2 || /** §8.2.1 Bullet 1 */
+                    $certProps['subject']['OU'] != "Authenticator Attestation" || /** §8.2.1 Bullet 2 [Subject-OU] */
+                    !isset($certProps['subject']['CN']) || /** §8.2.1 Bullet 2 [Subject-CN] */
                     !isset($certProps['extensions']['basicConstraints']) ||
                     strstr("CA:FALSE", $certProps['extensions']['basicConstraints']) === false                     /** §8.2.1 Bullet 4 */
             ) {
                 $this->fail("Attestation certificate properties are no good.");
             }
             if (isset(AAGUID::AAGUID_DICTIONARY[strtolower($this->AAGUID)])) {
-                if ($certProps['subject']['O'] != AAGUID::AAGUID_DICTIONARY[strtolower($this->AAGUID)]['O'] ||     /** §8.2.1 Bullet 2 [Subject-O] */
+                if ($certProps['subject']['O'] != AAGUID::AAGUID_DICTIONARY[strtolower($this->AAGUID)]['O'] || /** §8.2.1 Bullet 2 [Subject-O] */
                         $certProps['subject']['C'] != AAGUID::AAGUID_DICTIONARY[strtolower($this->AAGUID)]['C']) { /** §8.2.1 Bullet 2 [Subject-C] */
                     $this->fail("AAGUID does not match vendor data.");
                 }
@@ -288,18 +288,18 @@ 
         if (isset($this->credential[-2]) && sizeof($this->credential[-2]) == 32 
               &&
             isset($this->credential[-3]) && sizeof($this->credential[-3]) == 32) {
-            $publicKeyU2F = chr(4).$this->credential[-2].$this->credential[-3];
+            $publicKeyU2F = chr(4) . $this->credential[-2] . $this->credential[-3];
         } else {
             $this->fail("FIDO U2F attestation: the public key is not as expected.");
         }
         /**
          * §8.6 Verification Step 5: create verificationData
          */
-        $verificationData = chr(0).$this->rpIdHash.$this->clientDataHash.$this->credentialId.$publicKeyU2F;
+        $verificationData = chr(0) . $this->rpIdHash . $this->clientDataHash . $this->credentialId . $publicKeyU2F;
         /**
          * §8.6 Verification Step 6: verify signature
          */
-        if (openssl_verify($verificationData, $stmtDecoded['sig'],$attCert, OPENSSL_ALGO_SHA256) !== 1) {
+        if (openssl_verify($verificationData, $stmtDecoded['sig'], $attCert, OPENSSL_ALGO_SHA256) !== 1) {
             $this->fail("FIDO U2F Attestation verification failed.");
         } else {
             $this->pass("Successfully verified FIDO U2F signature.");

lib/Auth/Process/WebAuthn.php

Spacing +3 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
             $this->store = Store::parseStoreConfig($config['store']);
         } catch (\Exception $e) {
             Logger::error(
-                'webauthn: Could not create storage: '.
+                'webauthn: Could not create storage: ' .
                 $e->getMessage()
             );
         }
@@ -132,8 +132,8 @@ 
         assert(array_key_exists('metadata-set', $state['Source']));
 
         if (!array_key_exists($this->usernameAttrib, $state['Attributes'])) {
-            Logger::warning('webauthn: cannot determine if user needs second factor, missing attribute "'.
-                $this->usernameAttrib.'".');
+            Logger::warning('webauthn: cannot determine if user needs second factor, missing attribute "' .
+                $this->usernameAttrib . '".');
             return;
         }