simplesamlphp / simplesamlphp-module-webauthn

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/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='" .

src/Controller/WebAuthn.php

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

@@ -80,7 +80,7 @@ 
          * exception.
          * 
          */
-            'aaguid_whitelist' => [ ],
+            'aaguid_whitelist' => [],
         /*
          * Some authenticators are more equal than others. Apple TouchID and
          * FaceID set their AAGUID to all-zeroes so can't be whitelisted. But
@@ -93,11 +93,11 @@ 
          * 
          * https://webkit.org/blog/11312/meet-face-id-and-touch-id-for-the-web/
          */
-            'attestation_format_whitelist' => [ ],
+            'attestation_format_whitelist' => [],
         ],
         'policy_passwordless' => [
             'minimum_certification_level' => WebAuthnRegistrationEvent::FIDO_CERTIFIED_L1,
-            'aaguid_whitelist' => [ ],
+            'aaguid_whitelist' => [],
             'attestation_format_whitelist' => ['apple'],
         ],
         

src/Controller/AuthProcess.php

Spacing +3 added lines, -3 removed lines

@@ -160,7 +160,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
         );
@@ -216,7 +216,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.");
@@ -240,7 +240,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);
         }