Re: [jira] Commented: (DIRMINA-422) create a new loggingfilter that leverages SLF4J's MDC feauture

[Maarten Bosteels <mb...@gmail.com>]

Just to let you know that I am working on it. But lots of work right now ...
I agree with all your remarks, but I can't find a proper name for the
WrappingFilter.

Maarten

On 8/22/07, Trustin Lee (JIRA) <ji...@apache.org> wrote:
>
>
>     [
> https://issues.apache.org/jira/browse/DIRMINA-422?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12521652]
>
> Trustin Lee commented on DIRMINA-422:
> -------------------------------------
>
> A few comments:
>
> * We could use Runnable instead of introducing FilterAction.
> * Let's make IoFilter methods in WrappingFilter 'final'.
> * Very trivial thing but I'd prefer protected abstract to abstract
> protected.
> * The name 'WrappingFilter' doesn't represent what it does clearly.  What
> would be a better name?  We will also have to change wrapFilterAction to
> something else.
>
> > Depending on the relative position of ExecutorFilter and
> ProtocolCodecFilter
> > more than one MdcLoggingFilter must be added to the chain
> > (when you want the MDC to be set during encoding/decoding).
>
> We could simply change MdcLoggingFilter to MDCInjectionFilter and make it
> perform property injection only, and let LoggingFilter handle logging. (
> i.e. Separation of concern :) WDYT?
>
>
>
> > create a new loggingfilter that leverages SLF4J's MDC feauture
> > --------------------------------------------------------------
> >
> >                 Key: DIRMINA-422
> >                 URL: https://issues.apache.org/jira/browse/DIRMINA-422
> >             Project: MINA
> >          Issue Type: New Feature
> >          Components: Filter
> >            Reporter: Maarten Bosteels
> >            Assignee: Maarten Bosteels
> >            Priority: Minor
> >             Fix For: 2.0.0-M1
> >
> >
> > A logging filter that puts some basic info about the IoSession in the
> MDC (Mapped Diagnostic Context)
> > has the advantage that all logging events down the call stack include
> this info.
> > Even if those log events are generated by code that is not aware of MINA
> at all.
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
>