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/Auth/Source/Passwordless.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
                         $config,
                         'authsources[' . var_export($this->authId, true) . ']'
         );
-        $this->authnContextClassRef = $this->authSourceConfig->getOptionalString("authncontextclassref",'urn:rsa:names:tc:SAML:2.0:ac:classes:FIDO');
+        $this->authnContextClassRef = $this->authSourceConfig->getOptionalString("authncontextclassref", 'urn:rsa:names:tc:SAML:2.0:ac:classes:FIDO');
         $moduleConfig = Configuration::getOptionalConfig('module_webauthn.php')->toArray();
 
         $initialStateData = new StateData();

src/Controller/AuthProcess.php

Spacing +2 added lines, -2 removed lines

@@ -209,7 +209,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.");
@@ -233,7 +233,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']];
             // now properly return our final state to the framework
             Source::completeAuth($state);
         }

src/Controller/RegProcess.php

Spacing +2 added lines, -2 removed lines

@@ -177,7 +177,7 @@ 
 
         // did we get any client extensions?
         $isResidentKey = 0;
-        if (strlen($request->request->get('clientext')) > 0 && count(json_decode($request->request->get('clientext'), true)) > 0 ) {
+        if (strlen($request->request->get('clientext')) > 0 && count(json_decode($request->request->get('clientext'), true)) > 0) {
             $extensions = json_decode($request->request->get('clientext'), true);
             if ($extensions['credProps']['rk'] === true) {
                 $isResidentKey = 1;
@@ -216,7 +216,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='" .

tests/src/Controller/ManageTokenTest.php

Indentation +6 added lines, -6 removed lines

@@ -93,8 +93,8 @@ 
             public static function loadState(string $id, string $stage, bool $allowMissing = false): ?array
             {
                 return [
-			'FIDO2AuthSuccessful' => true,
-			'FIDO2PasswordlessAuthMode' => false,
+            'FIDO2AuthSuccessful' => true,
+            'FIDO2PasswordlessAuthMode' => false,
                 ];
             }
         });
@@ -154,8 +154,8 @@ 
             public static function loadState(string $id, string $stage, bool $allowMissing = false): ?array
             {
                 return [
-			'FIDO2AuthSuccessful' => true,
-			'FIDO2PasswordlessAuthMode' => false,
+            'FIDO2AuthSuccessful' => true,
+            'FIDO2PasswordlessAuthMode' => false,
                 ];
             }
         });
@@ -189,8 +189,8 @@ 
                     'FIDO2AuthSuccessful' => false,
                     'FIDO2Tokens' => [0 => "foo"],
                     'FIDO2WantsRegister' => false,
-		    'UseInflowRegistration' => false,
-		    'FIDO2PasswordlessAuthMode' => false,
+            'UseInflowRegistration' => false,
+            'FIDO2PasswordlessAuthMode' => false,
                 ];
             }
         });

tests/src/Controller/AuthProcessTest.php

Indentation +2 added lines, -2 removed lines

@@ -85,8 +85,8 @@
                     'FIDO2Scope' => 'Ducktown',
                     'FIDO2Tokens' => [],
                     'FIDO2SignupChallenge' => 'abc123',
-		    'FIDO2AuthSuccessful' => true,
-		    'FIDO2PasswordlessAuthMode' => false,
+            'FIDO2AuthSuccessful' => true,
+            'FIDO2PasswordlessAuthMode' => false,
                     'requestTokenModel' => 'something',
                 ];
             }

tests/src/Controller/WebAuthnTest.php

Indentation +10 added lines, -10 removed lines

@@ -53,11 +53,11 @@ 
             'simplesaml'
         );
 
-	$this->module_config = [];
-	$this->module_config = Configuration::loadFromArray(
-		[
-			'registration' => ['use_inflow_registration' => true],
-		]);
+    $this->module_config = [];
+    $this->module_config = Configuration::loadFromArray(
+        [
+            'registration' => ['use_inflow_registration' => true],
+        ]);
 
         $this->session = Session::getSessionFromRequest();
 
@@ -66,7 +66,7 @@ 
             {
                 // do nothing
             }
-	};
+    };
         Configuration::setPreLoadedConfig($this->config, 'config.php');
         Configuration::setPreLoadedConfig($this->module_config, 'module_webauthn.php');
     }
@@ -96,11 +96,11 @@ 
                     'FIDO2Scope' => 'Ducktown',
                     'FIDO2Tokens' => [0 => 'A1B2C3', 1 => 'D4E5F6'],
                     'FIDO2SignupChallenge' => 'A1B2C3',
-		    'FIDO2WantsRegister' => false,
-		    'FIDO2PasswordlessAuthMode' => false,
+            'FIDO2WantsRegister' => false,
+            'FIDO2PasswordlessAuthMode' => false,
                     'FIDO2AuthSuccessful' => false,
-		    'requestTokenModel' => 'something',
-		    'Source' => 'There is no real auth source in this test.',
+            'requestTokenModel' => 'something',
+            'Source' => 'There is no real auth source in this test.',
                 ];
             }
         });

src/Controller/WebAuthn.php

Indentation +3 added lines, -3 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;
         }
@@ -213,8 +213,8 @@ 
         $t->data['showExitButton'] = !array_key_exists('Registration', $state);
         $frontendData['usernameEncoded'] = $usernameEncoded;
         $frontendData['attestation'] = $state['requestTokenModel'] ? "indirect" : "none";
-	$frontendData['credentialIdEncoded'] = $credentialIdEncoded;
-	$frontendData['FIDO2PasswordlessAuthMode'] = $state['FIDO2PasswordlessAuthMode'];
+    $frontendData['credentialIdEncoded'] = $credentialIdEncoded;
+    $frontendData['FIDO2PasswordlessAuthMode'] = $state['FIDO2PasswordlessAuthMode'];
         $t->data['frontendData'] = json_encode($frontendData);
 
         $t->data['FIDO2AuthSuccessful'] = $state['FIDO2AuthSuccessful'];

Spacing +3 added lines, -3 removed lines

@@ -166,7 +166,7 @@ 
 
         $state = $this->authState::loadState($stateId, 'webauthn:request');
 
-        if ( $this->workflowStateMachine($state) != self::STATE_AUTH_NOMGMT ) {
+        if ($this->workflowStateMachine($state) != self::STATE_AUTH_NOMGMT) {
             $templateFile = 'webauthn:webauthn.twig';
         } else {
             $templateFile = 'webauthn:authentication.twig';
@@ -206,7 +206,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];
         }
 
@@ -218,7 +218,7 @@ 
         $t->data['frontendData'] = json_encode($frontendData);
 
         $t->data['FIDO2AuthSuccessful'] = $state['FIDO2AuthSuccessful'];
-        if ( $this->workflowStateMachine($state) == self::STATE_MGMT ) {
+        if ($this->workflowStateMachine($state) == self::STATE_MGMT) {
             $t->data['regURL'] = Module::getModuleURL('webauthn/regprocess?StateId=' . urlencode($stateId));
             $t->data['delURL'] = Module::getModuleURL('webauthn/managetoken?StateId=' . urlencode($stateId));
 

config-templates/module_webauthn.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
          * exception.
          * 
          */
-        'aaguid_whitelist' => [ ],
+        'aaguid_whitelist' => [],
         
         /*
          * Some authenticators are more equal than others. Apple TouchID and