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

ConfigurationDelegate.groovy should check AppenderAttachable not AsyncAppender

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 1.1.2
    • Fix Version/s: 1.1.3
    • Component/s: logback-classic
    • Labels:

      Description

      When you see ConfigurationDelegate.groovy's 136 line,

            AppenderDelegate ad = clazz.name == 'ch.qos.logback.classic.AsyncAppender' ? new AsyncAppenderDelegate(appender, appenderList) : new AppenderDelegate(appender);
      

      this line check the class is AsyncAppender.
      But there is another async appender like reactor-logback's AsyncAppdener.

      So this linke should changed to check if the class implements ch.qos.logback.core.spi.AppenderAttachable.

      Because of this, we cannot use reactor-logback with groovy configuration.

        Attachments

          Activity

            People

            • Assignee:
              tony19 Tony Trinh
              Reporter:
              kwon37xi KwonNam Son
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: