Skip to content

Commit 005a5ed

Browse files
Fix(#3239) NPE, for RegexFilter creator.
1 parent 8908092 commit 005a5ed

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

log4j-core-test/src/test/java/org/apache/logging/log4j/core/filter/RegexFilterTest.java

+8
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818

1919
import static org.hamcrest.CoreMatchers.equalTo;
2020
import static org.hamcrest.MatcherAssert.assertThat;
21+
import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
2122
import static org.junit.jupiter.api.Assertions.assertNull;
2223
import static org.junit.jupiter.api.Assertions.assertSame;
2324
import static org.junit.jupiter.api.Assertions.assertTrue;
@@ -39,6 +40,13 @@ static void before() {
3940
StatusLogger.getLogger().setLevel(Level.OFF);
4041
}
4142

43+
@Test
44+
void testRegexFilterDoesNotThrowWithAllTheParametersExceptRegexEqualNull() {
45+
assertDoesNotThrow(() -> {
46+
RegexFilter.createFilter(".* test .*", null, null, null, null);
47+
});
48+
}
49+
4250
@Test
4351
void testThresholds() throws Exception {
4452
RegexFilter filter = RegexFilter.createFilter(".* test .*", null, false, null, null);

log4j-core/src/main/java/org/apache/logging/log4j/core/filter/RegexFilter.java

+2-1
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,8 @@ public static RegexFilter createFilter(
148148
LOGGER.error("A regular expression must be provided for RegexFilter");
149149
return null;
150150
}
151-
return new RegexFilter(useRawMsg, Pattern.compile(regex, toPatternFlags(patternFlags)), match, mismatch);
151+
return new RegexFilter(
152+
Boolean.TRUE.equals(useRawMsg), Pattern.compile(regex, toPatternFlags(patternFlags)), match, mismatch);
152153
}
153154

154155
private static int toPatternFlags(final String[] patternFlags)

0 commit comments

Comments
 (0)