LOG4J2-3357 - use time trigger was scheduled to fire instead of current time.

Author: rgoers

log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/CronTriggeringPolicy.java

@@ -147,7 +147,8 @@ private static CronExpression getSchedule(final String expression) {
     }
 
     private void rollover() {
-		manager.rollover(cronExpression.getPrevFireTime(new Date()).getTime(), lastRollDate.getTime());
+        Date rollTime = future != null ? future.getFireTime() : new Date();
+		manager.rollover(cronExpression.getPrevFireTime(rollTime).getTime(), lastRollDate.getTime());
         if (future != null) {
             lastRollDate = future.getFireTime();
         }

src/changes/changes.xml

@@ -201,7 +201,10 @@
         Fixes incorrect constructor call in LocalizedMessageFactory.
       </action>
     </release>
-    <release version="2.19.1" date="TBD" description="GA Release 2.19.1">
+    <release version="2.20.0" date="TBD" description="GA Release 2.20.0">
+      <action issue="LOG4J2-3357" dev="rgoers" type="fix">
+        Use time trigger was expected to fire instead of current time.
+      </action>
       <action issue="LOG4J2-2297" dev="rgoers" type="fix">
         Allow java.sql.Time objects to format properly in MapMessages.
       </action>