Repository: logging-log4j2
Updated Branches:
  refs/heads/master 559e1de80 -> eb28033c5


The config object can be null when this builder is built
programmatically.


Project: http://git-wip-us.apache.org/repos/asf/logging-log4j2/repo
Commit: http://git-wip-us.apache.org/repos/asf/logging-log4j2/commit/eb28033c
Tree: http://git-wip-us.apache.org/repos/asf/logging-log4j2/tree/eb28033c
Diff: http://git-wip-us.apache.org/repos/asf/logging-log4j2/diff/eb28033c

Branch: refs/heads/master
Commit: eb28033c5845267d4b212759b0dabfbe586ac268
Parents: 559e1de
Author: Gary Gregory <[email protected]>
Authored: Sun Jul 15 08:17:33 2018 -0600
Committer: Gary Gregory <[email protected]>
Committed: Sun Jul 15 08:17:33 2018 -0600

----------------------------------------------------------------------
 .../log4j/core/appender/rolling/DefaultRolloverStrategy.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/logging-log4j2/blob/eb28033c/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
----------------------------------------------------------------------
diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
index dfacd72..e5298bc 100644
--- 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
+++ 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java
@@ -142,7 +142,9 @@ public class DefaultRolloverStrategy extends 
AbstractRolloverStrategy {
                 }
             }
             final int compressionLevel = 
Integers.parseInt(compressionLevelStr, Deflater.DEFAULT_COMPRESSION);
-            return new DefaultRolloverStrategy(minIndex, maxIndex, useMax, 
compressionLevel, config.getStrSubstitutor(),
+            // The config object can be null when this object is built 
programmatically.
+            StrSubstitutor nonNullStrSubstitutor = config != null ? 
config.getStrSubstitutor() : new StrSubstitutor();
+            return new DefaultRolloverStrategy(minIndex, maxIndex, useMax, 
compressionLevel, nonNullStrSubstitutor,
                     customActions, stopCustomActionsOnError, 
tempCompressedFilePattern);
         }
 

Reply via email to