Uploaded image for project: 'logback'
  1. logback
  2. LOGBACK-235

log rollover should be configurable using %d multiple times in file name pattern

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0.1
    • Component/s: logback-core
    • Labels:
      None

      Description

      Trying to configure a log rollover with logs being rolled daily and put into a directory named "yyyy-MM", i.e. one folder for each month containing that month's logs, I'm using the following appender config:

        <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
          <file>app.log</file>
      
          <rollingPolicy class="ch.qos.logback.core.rolling.TimeBasedRollingPolicy">
            <!-- rollover every minute -->
            <fileNamePattern%d{yyyy-MM}/app.%d{yyyy-MM-dd}.log</fileNamePattern>
          </rollingPolicy>
      
          <encoder>
            <pattern>%d{"yyyy-MM-dd HH:mm:ss,SSS"} [%thread] %-5level %logger{36} - %msg%n%xEx</pattern>
          </encoder>
        </appender>
      

      However, it seems as if the current logback implementation merely considers the first %d specification, which leads to a log rotation only once per month instead of the desired once per day frequency.

      I figure this is because via %d, the log rotation frequency and the file name pattern are somewhat tied together and seemingly can't be specified independently.

      It would be nice if either multiple %d specifiers would be supported, with an option to specify which one should be considered for the rollover period.
      Even better, it might be a good idea to support the rollover period to be specified separately and independently from the file name pattern; arguably, using the file name patterns for both is somewhat convenient, but violates the principle of "separation of concerns".

        Attachments

          Activity

            People

            • Assignee:
              ceki Ceki Gülcü
              Reporter:
              thomas.corte Thomas Corte
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: