simplesamlphp / simplesamlphp-module-webauthn

lib/WebAuthn/WebAuthnRegistrationEvent.php

Spacing +4 added lines, -4 removed lines

@@ -170,16 +170,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.");
                 }

www/authprocess.php

Spacing +2 added lines, -2 removed lines

@@ -73,10 +73,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: ".\SimpleSAML\Module::getModuleURL('webauthn/webauthn.php?StateId='.urlencode($id)));
+        header("Location: " . \SimpleSAML\Module::getModuleURL('webauthn/webauthn.php?StateId=' . urlencode($id)));
     } else {
         \SimpleSAML\Auth\ProcessingChain::resumeProcessing($state);
     }