Add support for max compression delay in compression actions

[ramanathan1504]

Fixes #4012

Changes

• Added maxCompressionDelay parameter to GzCompressAction and ZipCompressAction
• Extended FileExtension to support max compression delay configuration
• Updated DefaultRolloverStrategy to pass compression delay settings to compression actions
• Added backward-compatible constructors for existing code
• Comprehensive test coverage for compression delay functionality

Files Modified

• Core Implementation

  • log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/FileExtension.java
  • log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/action/GzCompressAction.java
  • log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/action/ZipCompressAction.java
  • log4j-core/src/main/java/org/apache/logging/log4j/core/appender/rolling/DefaultRolloverStrategy.java

• Tests

  • log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/rolling/FileExtensionCompressDelayTest.java (new)
  • log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/GzCompressActionTest.java (new)
  • log4j-core-test/src/test/java/org/apache/logging/log4j/core/appender/rolling/action/ZipCompressActionTest.java (new)

[vy]

@ramanathan1504, thanks for taking the time. See my remarks and please add a changelog entry.

TBH, I find your submission very sloppy in the sense that it appears to be generated using an LLM prompt and then without a review from your side pushed on us for review. Please, don't do this. Review your changes before asking review from others.

[ramanathan1504]

@ramanathan1504, thanks for taking the time. See my remarks and please add a changelog entry.

TBH, I find your submission very sloppy in the sense that it appears to be generated using an LLM prompt and then without a review from your side pushed on us for review. Please, don't do this. Review your changes before asking review from others.

ok next time i will aware about this.. i think its happened couple of time..thanks

[vy]

@ramanathan1504, you need to move @since and @Version tags to 2.27.0, FYI.

[ramanathan1504]

@ramanathan1504, you need to move @since and @Version tags to 2.27.0, FYI.

ok

[vy]

@ramanathan1504, could you update the documentation to reflect the newly added maxCompressionDelaySeconds configuration attribute, please?

[ramanathan1504]

@ramanathan1504, could you update the documentation to reflect the newly added maxCompressionDelaySeconds configuration attribute, please?

sure i will add that 👍🏾