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/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/Store.php

Spacing +1 added lines, -1 removed lines

@@ -152,7 +152,7 @@
      *
      * @throws \Exception if the configuration is invalid.
      */
-    public static function parseStoreConfig(string|array $config): Store
+    public static function parseStoreConfig(string | array $config): Store
     {
         if (is_string($config)) {
             $arrayUtils = new Utils\Arrays();

src/Controller/AuthProcess.php

Spacing +4 added lines, -4 removed lines

@@ -154,7 +154,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,
         );
@@ -217,7 +217,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'] ?
@@ -248,7 +248,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.
 
@@ -266,7 +266,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/Controller/WebAuthn.php

Indentation +1 added lines, -1 removed lines

@@ -101,7 +101,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;

Spacing +2 added lines, -2 removed lines

@@ -113,7 +113,7 @@ 
     }
 
 
-    public static function loadModuleConfig(array $moduleConfig, StateData &$stateData): void
+    public static function loadModuleConfig(array $moduleConfig, StateData & $stateData): void
     {
         $stateData->store = Store::parseStoreConfig($moduleConfig['store']);
 
@@ -219,7 +219,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/Auth/Process/WebAuthn.php

Indentation +2 added lines, -2 removed lines

@@ -151,8 +151,8 @@
         $session = Session::getSessionFromRequest();
         $lastSecondFactor = $session->getData("DateTime", 'LastSuccessfulSecondFactor');
         if // do we need to do secondFactor in interval, or even every time?
-           // we skip only if an interval is configured AND we did successfully authenticate,
-           // AND are within the interval
+            // we skip only if an interval is configured AND we did successfully authenticate,
+            // AND are within the interval
         (
                 $this->SecondFactorMaxAge >= 0 && $lastSecondFactor instanceof \DateTime
         ) {

src/WebAuthn/WebAuthnRegistrationEvent.php

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
     public const FIDO_CERTIFIED_L3PLUS = "FIDO_CERTIFIED_L3plus";
 
     // 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",
@@ -755,7 +755,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) .
                 ").");