Fixes a regex expression bug. When the left-hand side of the operatio…

…n is null, always return false rather than throwing an exception. Resolves #4763. Signed-off-by: David Venable <dlv@amazon.com>
opensearch-project · Aug 1, 2024 · 25e2fc0 · 25e2fc0
1 parent fd87e3d
commit 25e2fc0
Show file tree

Hide file tree

Showing 4 changed files with 123 additions and 104 deletions.
diff --git a/...ession/src/main/java/org/opensearch/dataprepper/expression/GenericRegexMatchOperator.java b/...ession/src/main/java/org/opensearch/dataprepper/expression/GenericRegexMatchOperator.java
@@ -37,6 +37,8 @@ public int getSymbol() {
     @Override
     public Boolean evaluate(final Object ... args) {
         checkArgument(args.length == 2, displayName + " requires operands length needs to be 2.");
+        if(args[0] == null)
+            return false;
         checkArgument(args[0] instanceof String, displayName + " requires left operand to be String.");
         checkArgument(args[1] instanceof String, displayName + " requires right operand to be String.");
         try {