ok, I will adapt the javadocs, I thought they were clear...

Can you have a look at the test-case for the MdcInjectingFilter, and see if
anything is missing ?

Thanks,
Maarten

On Fri, Feb 29, 2008 at 10:58 PM, Niklas Gustavsson <[EMAIL PROTECTED]>
wrote:

> On Fri, Feb 29, 2008 at 10:52 PM, Mike Heath <[EMAIL PROTECTED]> wrote:
> >  The MDC properties are stored in a ThreadLocal so if you have the
> >  logging filter before an executor filter, the MDC properties will get
> >  lost.  If you put the MDC logging filter after the executor filter then
> >  it should work just fine.
>
> Seconds after sending my email I realized that it was the same
> MdcInjectionFilter instance needed in both places, previously we
> created separate instances. This fixes the issue Dave discovered. Fix
> commited in rev 632454.
>
> The JavaDoc doesn't make this very clear and I'm not sure if it is the
> intended behavior?
>
> /niklas
>

Reply via email to