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

ConfigurationDelegate.groovy should check AppenderAttachable not AsyncAppender

    XMLWordPrintable

Details

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • 1.1.3
    • 1.1.2
    • logback-classic

    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

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

            Dates

              Created:
              Updated:
              Resolved: