Improve AuthenticatorStatusValidators logging

OpenConext · Apr 16, 2024 · 530294e · 530294e
1 parent 331a33b
commit 530294e
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/src/Service/AuthenticatorStatusValidator.php b/src/Service/AuthenticatorStatusValidator.php
@@ -56,18 +56,22 @@ public function validate(array $statusReports): void
     {
         $meetsRequirement = false;
         $reportsProcessed = 0;
+        $reportLog = [];
         foreach ($statusReports as $report) {
             if (in_array($report->status, $this->allowedStatus)) {
                 $meetsRequirement = true;
             }
             $reportsProcessed++;
+            $reportLog[] = $report->status;
         }
 
         if (!$meetsRequirement) {
             throw new AuthenticatorStatusNotSupportedException(
                 sprintf(
-                    'Of the %d StatusReports tested, none met one of the required FIDO Certified statuses',
-                    $reportsProcessed
+                    'Of the %d StatusReports tested, none met one of the required FIDO Certified statuses. ' .
+                    'Reports tested: "%s"',
+                    $reportsProcessed,
+                    implode(', ', $reportLog)
                 )
             );
         }