Hi!
Borut Bolčina <borut.bolc...@gmail.com> 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
