Improve log integration for elasticearch

Not all queries have 'took'. Account for that and use defaults values. Fixes #989
cakephp · Jan 25, 2024 · 860b504 · 860b504
1 parent cdb3138
commit 860b504
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/src/Database/Log/DebugLog.php b/src/Database/Log/DebugLog.php
@@ -135,15 +135,16 @@ public function log($level, string|Stringable $message, array $context = []): vo
 
         // This specific to Elastic Search
         if (!$query instanceof LoggedQuery && isset($context['request']) && isset($context['response'])) {
-            $this->_totalTime += $context['response']['took'];
+            $took = $context['response']['took'] ?? 0;
+            $this->_totalTime += $took;
 
             $this->_queries[] = [
                 'query' => json_encode([
                     'method' => $context['request']['method'],
                     'path' => $context['request']['path'],
                     'data' => $context['request']['data'],
                 ], JSON_PRETTY_PRINT),
-                'took' => $context['response']['took'] ?: 0,
+                'took' => $took ?: 0,
                 'rows' => $context['response']['hits']['total']['value'] ?? $context['response']['hits']['total'] ?? 0,
             ];