Do like this:
MDC.put("xxx", ...);
MDC.put("yyy", ...);
try {
return handler.service(request, response);
} finally {
MDC.remove("xxx")
MDC.remove("yyy")
}
or else you will have problems when your request throws an exception....
Olle Hallin
Senior Java Developer and Architect
[email protected]
www.crisp.se
2009/10/9 <[email protected]>
> Hi!
>
> Borut Bolčina <[email protected]> schrieb am 09.10.2009 14:55:32:
>
> > this is what I did:
> >
> > public class RequestLoggingFilter implements HttpServletRequestFilter {
> > public final RequestGlobals requestGlobals;
> >
> > public RequestLoggingFilter(final RequestGlobals requestGlobalss) {
> > this.requestGlobals = requestGlobalss;
> > }
> >
> > @Override
> > public boolean service(HttpServletRequest request,
> HttpServletResponse
> > response, HttpServletRequestHandler handler)
> > throws IOException {
> > MDC.put("remoteIP",
> > this.requestGlobals.getHTTPServletRequest().getRemoteAddr());
> >
> > String s =
> > this.requestGlobals.getHTTPServletRequest().getRequestedSessionId();
> > if (s == null) {
> > s = "";
> > }
> > MDC.put("sessionID", s);
> > return handler.service(request, response);
> > }
> > }
>
> >
> > but I feel I am missing something. Where do I put the code:
> > MDC.remove("remoteIP");
> > MDC.remove("sessionID");
> >
>
> In a similar situation, I used something like
> --------------------
> @Override
> public boolean service(HttpServletRequest request, HttpServletResponse
> response, HttpServletRequestHandler handler)
> throws IOException {
> MDC.put("remoteIP",
> this.requestGlobals.getHTTPServletRequest().getRemoteAddr());
>
> String s =
> this.requestGlobals.getHTTPServletRequest().getRequestedSessionId();
> if (s == null) {
> s = "";
> }
> MDC.put("sessionID", s);
> boolean result = handler.service(request, response);
>
> MDC.remove("remoteIP");
> MDC.remove("sessionID");
>
> return result;
> }
> }
> --------------------
>
> but I'm not sure if it does the right thing in all situations.
>
> Dirk
>
>
>
> BGS Beratungsgesellschaft
> Software Systemplanung AG
>
>
>
>
> Niederlassung Köln/Bonn
> Grantham-Allee 2-8
> 53757 Sankt Augustin
> Fon: +49 (0) 2241 / 166-500
> Fax: +49 (0) 2241 / 166-680
> www.bgs-ag.de
> Geschäftssitz Mainz
> Registergericht
> Amtsgericht Mainz
> HRB 62 50
>
> Aufsichtsratsvorsitzender
> Klaus Hellwig
> Vorstand
> Hermann Kiefer
> Nils Manegold
> Thomas Reitz
>
>
