bug #50913 [HttpKernel][WebProfilerBundle] Fix search feature (Cyril HERRERA)

nicolas-grekas · nicolas-grekas · commit e8dc1c2b0c29 · 2023-07-13T15:28:51.000+02:00
This PR was squashed before being merged into the 5.4 branch. Discussion ---------- [HttpKernel][WebProfilerBundle] Fix search feature | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | - | License | MIT | Doc PR | - This fix research feature in the profiler. Sometimes lines in index.csv are broken with something like this : ,,200 the try and catch allow the profiler to skip these lines Commits ------- cffceb1 [HttpKernel][WebProfilerBundle] Fix search feature
diff --git a/src/Symfony/Component/HttpKernel/Profiler/FileProfilerStorage.php b/src/Symfony/Component/HttpKernel/Profiler/FileProfilerStorage.php
@@ -61,6 +61,12 @@ public function find(?string $ip, ?string $url, ?int $limit, ?string $method, in
         $result = [];
         while (\count($result) < $limit && $line = $this->readLineFromFile($file)) {
             $values = str_getcsv($line);
+
+            if (7 !== \count($values)) {
+                // skip invalid lines
+                continue;
+            }
+
             [$csvToken, $csvIp, $csvMethod, $csvUrl, $csvTime, $csvParent, $csvStatusCode] = $values;
             $csvTime = (int) $csvTime;
 
