From 7747be768fdb788f7901599c401b4237e60112c0 Mon Sep 17 00:00:00 2001 From: Shizuo Fujita Date: Thu, 23 Jul 2026 11:40:46 +0900 Subject: [PATCH] Fix outdated regexp of apache2 parser The regexp shown in parser/apache2.md was the one used before v1.14.2. Escape sequences were first allowed by df5ac352 "Allow escape sequences in Apache access log" (v0.14.14), which used `(?:[^\"]|\\.)`, and that is the form the document still shows. The source was later narrowed by db93f4b9 "parser_apache2: Fix too wide matching regexp" (v1.14.2): - REGEXP = ... (?(?:[^\"]|\\.)*?) ... (?(?:[^\"]|\\.)*) ... (?(?:[^\"]|\\.)*) ... + REGEXP = ... (?(?:[^\"]|\\")*?) ... (?(?:[^\"]|\\")*) ... (?(?:[^\"]|\\")*) ... The document was not updated at that time. So it still says that any backslash-escaped character is accepted in path, referer and agent, while the current implementation accepts only an escaped double quote. The new text is copied from Apache2Parser::REGEXP.source of the v1.19 branch and was compared with the document line to confirm they are identical. Signed-off-by: Shizuo Fujita --- parser/apache2.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/parser/apache2.md b/parser/apache2.md index c707bd2f..de625614 100644 --- a/parser/apache2.md +++ b/parser/apache2.md @@ -11,7 +11,7 @@ See [Parse Section Configurations](../configuration/parse-section.md) Here is the regexp and time format patterns of this plugin: ```text -expression /^(?[^ ]*) [^ ]* (?[^ ]*) \[(?