[logging-log4j2] branch release-2.x updated: Trim input to Integer.parseInt() in DefaultRolloverStrategy. (#755)

[vy]

This is an automated email from the ASF dual-hosted git repository.

vy pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/release-2.x by this push:
     new 92c73ed  Trim input to Integer.parseInt() in DefaultRolloverStrategy. 
(#755)
92c73ed is described below

commit 92c73edbdece974429643ec541454b73e3dde391
Author: Volkan Yazici <vol...@yazi.ci>
AuthorDate: Wed Feb 16 10:59:11 2022 +0100

    Trim input to Integer.parseInt() in DefaultRolloverStrategy. (#755)
---
 .../logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

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 43a789f..d747c83 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
@@ -126,7 +126,7 @@ public class DefaultRolloverStrategy extends 
AbstractRolloverStrategy {
                 useMax = fileIndex == null ? true : 
fileIndex.equalsIgnoreCase("max");
                 minIndex = MIN_WINDOW_SIZE;
                 if (min != null) {
-                    minIndex = Integer.parseInt(min);
+                    minIndex = Integer.parseInt(min.trim());
                     if (minIndex < 1) {
                         LOGGER.error("Minimum window size too small. Limited 
to " + MIN_WINDOW_SIZE);
                         minIndex = MIN_WINDOW_SIZE;
@@ -141,7 +141,7 @@ public class DefaultRolloverStrategy extends 
AbstractRolloverStrategy {
                     }
                 }
             }
-            final int compressionLevel = 
Integers.parseInt(compressionLevelStr, Deflater.DEFAULT_COMPRESSION);
+            final int compressionLevel = 
Integers.parseInt(compressionLevelStr.trim(), Deflater.DEFAULT_COMPRESSION);
             // The config object can be null when this object is built 
programmatically.
             final StrSubstitutor nonNullStrSubstitutor = config != null ? 
config.getStrSubstitutor() : new StrSubstitutor();
                        return new DefaultRolloverStrategy(minIndex, maxIndex, 
useMax, compressionLevel, nonNullStrSubstitutor,