Преглед на файлове

Remove redundant nested operator in builtin grok expression. (#47870)

This prevents the following warning from being printed to console:
`regular expression has redundant nested repeat operator + /%\{(?<name>(?<pattern>[A-z0-9]+)(?::(?<subname>[[:alnum:]@\[\]_:.-]+))?)(?:=(?<definition>(?:(?:[^{}]+|\.+)+)+))?\}/`

The current grok expression is not failing, but just this warning is being printed.
The warning started being printed after upgrading joni (#47374).

Closes #47861
Martijn van Groningen преди 6 години
родител
ревизия
9e7cfc8183
променени са 1 файла, в които са добавени 0 реда и са изтрити 2 реда
  1. 0 2
      libs/grok/src/main/java/org/elasticsearch/grok/Grok.java

+ 0 - 2
libs/grok/src/main/java/org/elasticsearch/grok/Grok.java

@@ -54,9 +54,7 @@ public final class Grok {
             "(?::(?<subname>[[:alnum:]@\\[\\]_:.-]+))?" +
             ")" +
             "(?:=(?<definition>" +
-            "(?:" +
             "(?:[^{}]+|\\.+)+" +
-            ")+" +
             ")" +
             ")?" + "\\}";
     private static final Regex GROK_PATTERN_REGEX = new Regex(GROK_PATTERN.getBytes(StandardCharsets.UTF_8), 0,