Skip to content

TUL/Add warn.log appender (A4) with gzip rotation#1283

Merged
milanmajchrak merged 1 commit intocustomer/TULfrom
propagate/reduce-logs-TUL
Mar 17, 2026
Merged

TUL/Add warn.log appender (A4) with gzip rotation#1283
milanmajchrak merged 1 commit intocustomer/TULfrom
propagate/reduce-logs-TUL

Conversation

@milanmajchrak
Copy link
Collaborator

Problem description

Added A4 appender for warn.log with daily gzip rotation to match configuration from other customer branches. Added AppenderRef at WARN level to DSpace logger and Root.

Analysis

(Write here, if there is needed describe some specific problem. Erase it, when it is not needed.)

Problems

(Write here, if some unexpected problems occur during solving issues. Erase it, when it is not needed.)

Manual Testing (if applicable)

Copilot review

  • Requested review from Copilot

Copilot AI review requested due to automatic review settings March 17, 2026 16:34
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the default Log4j2 configuration (dspace/config/log4j2.xml) to introduce a dedicated rolling log file for WARN+ events and route warnings/errors to it, separating them from the main dspace.log.

Changes:

  • Add a new RollingFile appender (A4) writing to warn.log with daily rollover.
  • Route WARN+ events from org.dspace and the Root logger to A4 via AppenderRef level="WARN".

You can also share your feedback on Copilot code review. Take the survey.

Added A4 appender for warn.log with daily gzip rotation to match
configuration from other customer branches. Added AppenderRef at
WARN level to DSpace logger and Root.
@milanmajchrak milanmajchrak force-pushed the propagate/reduce-logs-TUL branch from dee0866 to 854c5e4 Compare March 17, 2026 16:37
@milanmajchrak milanmajchrak merged commit ccdbf8f into customer/TUL Mar 17, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants