Fix monolog line parser to avoid breaking on json with [ and { combinations (#215)

Author: frankdekker

README.md

@@ -100,7 +100,7 @@ fd_log_viewer:
             downloadable: false
             deletable: false
             start_of_line_pattern: '/^\[\d{4}-\d{2}-\d{2}[^]]*]\s+\S+\.\S+:/'
-            log_message_pattern: '/^\[(?P<date>[^\]]+)\]\s+(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+(?P<message>.*)\s+(?P<context>[[{].*?[\]}])\s+(?P<extra>[[{].*?[\]}])\s+$/s'
+            log_message_pattern: '/^\[(?P<date>[^\]]+)\]\s+(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+(?P<message>.*)\s+(?P<context>(?:{.*?}|\[.*?]))\s+(?P<extra>(?:{.*?}|\[.*?]))\s+$/s'
             date_format: null
 
     hosts:

docs/configuration-reference.md

@@ -21,7 +21,7 @@ fd_log_viewer:
             downloadable: false
             deletable: false
             start_of_line_pattern: '/^\[\d{4}-\d{2}-\d{2}[^]]*]\s+\S+\.\S+:/'
-            log_message_pattern: '/^\[(?P<date>[^\]]+)\]\s+(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+(?P<message>.*)\s+(?P<context>[[{].*?[\]}])\s+(?P<extra>[[{].*?[\]}])\s+$/s'
+            log_message_pattern: '/^\[(?P<date>[^\]]+)\]\s+(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+(?P<message>.*)\s+(?P<context>(?:{.*?}|\[.*?]))\s+(?P<extra>(?:{.*?}|\[.*?]))\s+$/s'
             date_format: null
 
     hosts:
@@ -172,7 +172,7 @@ This regex pattern is used to parse the log entry into the following named group
 - `extra`
 
 The default monolog regex
-pattern `/^\[(?P<date>[^\]]+)\]\s+(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+(?P<message>.*)\s+(?P<context>[[{].*?[\]}])\s+(?P<extra>[[{].*?[\]}])\s+$/s`
+pattern `/^\[(?P<date>[^\]]+)\]\s+(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+(?P<message>.*)\s+(?P<context>(?:{.*?}|\[.*?]))\s+(?P<extra>(?:{.*?}|\[.*?]))\s+$/s`
 matches:
 
 ```text

src/Service/File/Monolog/MonologLineParser.php

@@ -16,8 +16,8 @@ class MonologLineParser implements LogLineParserInterface
         '/^\[(?P<date>[^\]]+)\]\s+' .
         '(?P<channel>[^\.]+)\.(?P<severity>[^:]+):\s+' .
         '(?P<message>.*)\s+' .
-        '(?P<context>[[{].*?[\]}])\s+' .
-        '(?P<extra>[[{].*?[\]}])\s+$/s';
+        '(?P<context>(?:{.*?}|\[.*?]))\s+' .
+        '(?P<extra>(?:{.*?}|\[.*?]))\s+$/s';
 
     private readonly string $logLinePattern;
 

tests/Unit/Service/File/Monolog/MonologLineParserTest.php

@@ -42,11 +42,11 @@ public function testParseWithContextAndExtra(): void
             'severity' => 'DEBUG',
             'channel'  => 'app',
             'message'  => 'message',
-            'context'  => ['context' => 'context'],
-            'extra'    => ['extra' => 'extra'],
+            'context'  => ['context' => '[object]context'],
+            'extra'    => ['extra' => '[object]extra'],
         ];
 
-        $result = $this->parser->parse('[2000-01-01T00:00:00.000] app.DEBUG: message {"context":"context"} {"extra":"extra"}' . "\n");
+        $result = $this->parser->parse('[2000-01-01T00:00:00.000] app.DEBUG: message {"context":"[object]context"} {"extra":"[object]extra"}' . "\n");
         static::assertSame($expected, $result);
     }