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

ConfigurationDelegate.groovy should check AppenderAttachable not AsyncAppender

    XMLWordPrintable

    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: