On Jan 25, 2008 5:55 PM, Maarten Bosteels <[EMAIL PROTECTED]> wrote: > On Jan 25, 2008 5:32 PM, Emmanuel Lecharny <[EMAIL PROTECTED]> wrote: > > > Trustin Lee wrote: > > > He will not mind (TM) :D > > > > > > So... now we can start to drop IoSessionLogger and you could add JUL > > > MDC formatter into org.apache.mina.util, right? > > > > > Wrong, unless I misunterpreted the meaning of JUL MDC. > > > > What are we exactly talking about ? We are not supposed to use JUL, are > > we ? > > > Emmanuel, let me recap what has been discussed before. > > 1) First we had the IoSessionLogger: it can prepend a session-identifier > (eg remote addr) to the log-message > 2) Then the MdcInjectionFilter was added (it has a very similar goal but > uses the MDC mechanism) > 3) I proposed to remove IoSessionLogger since we now have the more > powerful MdcInjectionFilter > but one problem is that java.util.logging is lacking the MDC feature > 4) then Ceki has added MDC support for jul to SLF4J but now problem > remains that there are now jul formatters >
correction: "there are now jul formatters" should have been "there ar NO jul formatters" > > that are MDC aware (except the one that I attached to the SLF4J bugzilla > ticket, but that one won't be released by SLF4J) > 5) Trustin suggested that we add some MDC-aware formatters for JUL to > org.apache.mina.util > before we remove IoSessionLogger for good. > > The MDC-aware formatters are 100% optional, it's just to provide jul users > a means to use the MdcInjectionFilter. > And users of log4j or logback can already use MdcInjectionFilter and that > won't change at all. > > I hope that clears things up ? > > Maarten >