Merge pull request #2 from mbarbey/master

fix issue when there is more than one U2F key registered
Samyoul · Oct 26, 2018 · 0625202 · 0625202
2 parents 87c1bbf + eef29a8
commit 0625202
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/U2FServer.php b/src/U2FServer.php
@@ -196,6 +196,7 @@ public static function register(RegistrationRequest $request, $response, $attest
     public static function makeAuthentication(array $registrations, $appId)
     {
         $signatures = [];
+        $challenge = static::createChallenge();
         foreach ($registrations as $reg) {
             if( !is_object( $reg ) ) {
                 throw new \InvalidArgumentException('$registrations of makeAuthentication() method only accepts array of object.');
@@ -204,7 +205,7 @@ public static function makeAuthentication(array $registrations, $appId)
             $signatures[] = new SignRequest([
                 'appId' => $appId,
                 'keyHandle' => $reg->keyHandle,
-                'challenge' => static::createChallenge(),
+                'challenge' => $challenge,
             ]);
         }
         return $signatures;
@@ -425,4 +426,4 @@ private static function fixSignatureUnusedBits($cert)
         return $cert;
     }
 
-}
+}