Heres what I have put together, use at your own risk. any comments welcome.
HttpServletResponseLoggingFilter.java
public class HttpServletResponseLoggingFilter implements Filter
{
private Logger mLogger =
Logger.getLogger(HttpServletResponseLoggingFilter.class);
public void doFilter(ServletRequest servletRequest,
ServletResponse servletResponse, FilterChain filterChain) throws
IOException, ServletException
{
if ( mLogger.isDebugEnabled() && servletResponse instanceof
HttpServletResponse &&
isNonStaticResource((HttpServletRequest)servletRequest) )
{
final LoggingHttpServletResponse
loggingHttpServletResponse = new
LoggingHttpServletResponse((HttpServletResponse)servletResponse);
try
{
mLogger.debug("Filtering request : " +
getFullRequestUrl((HttpServletRequest)servletRequest));
filterChain.doFilter(servletRequest,
loggingHttpServletResponse);
}
finally
{
loggingHttpServletResponse.finishResponse();
}
}
else
{
filterChain.doFilter(servletRequest, servletResponse);
}
}
private boolean isNonStaticResource(HttpServletRequest request)
{
return !request.getRequestURI().contains("resources");
}
// http://hostname.com/mywebapp/servlet/MyServlet/a/b;c=123?d=789
private String getFullRequestUrl(HttpServletRequest request)
{
String reqUrl = request.getRequestURL().toString();
String queryString = request.getQueryString(); // d=789
if (queryString != null)
{
reqUrl += "?"+queryString;
}
return reqUrl;
}
public void init(FilterConfig filterConfig) throws ServletException
{
}
public void destroy()
{
}
}
LoggingHttpServletResponse.java
class LoggingHttpServletResponse extends HttpServletResponseWrapper
{
private Logger mLogger = Logger.getLogger(LoggingHttpServletResponse.class);
public LoggingHttpServletResponse(HttpServletResponse httpServletResponse)
{
super(httpServletResponse);
mWrappedResponse = httpServletResponse;
}
// ----------------------------------------------------- Instance Variables
/**
* Original response
*/
protected HttpServletResponse mWrappedResponse = null;
/**
* The ServletOutputStream that has been returned by
* <code>getOutputStream()</code>, if any.
*/
protected ServletOutputStream mStream = null;
/**
* The PrintWriter that has been returned by
* <code>getWriter()</code>, if any.
*/
protected PrintWriter mWriter = null;
// --------------------------------------------------------- Public Methods
/**
* Create and return a ServletOutputStream to write the content
* associated with this Response.
*
* @throws IOException if an input/output error occurs
*/
public ServletOutputStream createOutputStream() throws IOException
{
mLogger.debug("Creating new LoggingOutputStream");
return new LoggingServletOutputStream(mWrappedResponse, mLogger);
}
/**
* Finish a response.
*/
public void finishResponse()
{
try
{
if (mWriter != null)
{
mWriter.close();
}
else
{
if (mStream != null)
mStream.close();
}
}
catch (IOException e)
{
}
}
// ------------------------------------------------ ServletResponse Methods
/**
* Flush the buffer and commit this response.
*
* @throws IOException if an input/output error occurs
*/
public void flushBuffer() throws IOException
{
mStream.flush();
}
/**
* Return the servlet output mStream associated with this Response.
*
* @throws IllegalStateException if <code>getWriter</code> has
* already been called for this response
* @throws IOException if an input/output error occurs
*/
public ServletOutputStream getOutputStream() throws IOException
{
if (mWriter != null)
throw new IllegalStateException("getWriter() has already
been called for this response");
if (mStream == null)
mStream = createOutputStream();
mLogger.debug("mStream is set to " + mStream + " in getOutputStream");
return (mStream);
}
/**
* Return the mWriter associated with this Response.
*
* @throws IllegalStateException if <code>getOutputStream</code> has
* already been called for this response
* @throws IOException if an input/output error occurs
*/
public PrintWriter getWriter() throws IOException
{
if (mWriter != null)
return (mWriter);
if (mStream != null)
throw new IllegalStateException("getOutputStream() has
already been called for this response");
mStream = createOutputStream();
mLogger.debug("mStream is set to " + mStream + " in getOutputStream");
// HttpServletResponse.getCharacterEncoding() shouldn't return null
// according the spec, so feel free to remove that "if"
mWriter = new PrintWriter(mStream);
return (mWriter);
}
}
LoggingServletOutputStream.java
class LoggingServletOutputStream extends ServletOutputStream
{
private Logger mLogger;
private HttpServletResponse mResponse;
private OutputStream mOutputStream;
private ByteArrayOutputStream mByteArrayOutputStream = new
ByteArrayOutputStream();
public LoggingServletOutputStream(HttpServletResponse response,
Logger logger) throws IOException
{
mResponse = response;
mOutputStream = mResponse.getOutputStream();
mLogger = logger;
}
public void write(int b) throws IOException
{
mByteArrayOutputStream.write(b);
mOutputStream.write(b);
}
@Override
public void write(byte b[]) throws IOException
{
mByteArrayOutputStream.write(b);
mOutputStream.write(b);
}
@Override
public void write(byte b[], int off, int len) throws IOException
{
mByteArrayOutputStream.write(b, off, len);
mOutputStream.write(b, off, len);
}
@Override
public void close() throws IOException
{
if ( mLogger.isDebugEnabled() )
{
float kBytes = mByteArrayOutputStream.size() / 1024f;
mLogger.debug("Writing " + kBytes + " kb (" +
mByteArrayOutputStream.size() +" b) to the client.\n" +
mByteArrayOutputStream.toString());
}
mByteArrayOutputStream = null;
mOutputStream.close();
}
@Override
public void flush() throws IOException
{
mOutputStream.flush();
}
}
web.xml Snippit..
<filter>
<filter-name>Logging Filter</filter-name>
<filter-class>com.nexusalpha.journeycheck.presentation.debug.HttpServletResponseLoggingFilter</filter-class>
</filter>
<filter-mapping>
<filter-name>Logging Filter</filter-name>
<url-pattern>/*</url-pattern>
</filter-mapping>
On 7/24/07, ben short <[EMAIL PROTECTED]> wrote:
Yes sure.
On 7/24/07, Karel V Sedlacek <[EMAIL PROTECTED]> wrote:
> Ben,
>
> When you succeed at this would you pass along your code? We have issues
> with timeouts and it would be great to see what's being passed along to
> the client.
>
> Karel
> Cornell University
>
> > Yes from the CompressionServletResponseWrapper example I can see the
> > methods I need to override as you have pointed out.
> >
> > On 7/24/07, Tim Funk <[EMAIL PROTECTED]> wrote:
> >> Yes - but tomcat doesn't have that functionality out of the box - you'd
> >> need to write a filter which creates a HttpServletRequestWrapper which
> >> overrides getOutputStream() (or getWriter()) and then passes back a
> >> wrapped OutStream or Writer which also logs to wherever when print(int
> >> i) is called.
> >>
> >> -Tim
> >>
> >> ben short wrote:
> >> > Hi Tim,
> >> >
> >> > Thanks for that, but it only seems to log out the request/response
> >> > headers. Is It possible to log everything sent to the client?
> >> >
> >> > Ben
> >> >
> >> > On 7/24/07, Tim Funk <[EMAIL PROTECTED]> wrote:
> >> >> Look at the RequestDumperValve
> >> >>
> >> >> -Tim
> >> >>
> >> >> ben short wrote:
> >> >> > Hi,
> >> >> >
> >> >> > I using Tomcat 6.0.13 and Spring 2.0.6. I have been involved in
> >> >> > developing a website that products pages in various formats , such
> >> as
> >> >> > www, xml, wap and pda. We are having some issues with wap and pda,
> >> but
> >> >> > cant ciew the html source thats being shown on the devices.
> >> >> > We can view the html source in firefox using a wap and pda plugin,
> >> but
> >> >> > the issues are not always the same or there at all.
> >> >> >
> >> >> > What I would like to do is log all the data sent to the client. I
> >> have
> >> >> > been looking at encapsulating the HttpServletRequest and log out
> >> the
> >> >> > data to a log file.
> >>
> >> ---------------------------------------------------------------------
> >> To start a new topic, e-mail: [email protected]
> >> To unsubscribe, e-mail: [EMAIL PROTECTED]
> >> For additional commands, e-mail: [EMAIL PROTECTED]
> >>
> >>
> >
> > ---------------------------------------------------------------------
> > To start a new topic, e-mail: [email protected]
> > To unsubscribe, e-mail: [EMAIL PROTECTED]
> > For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
>
>
>
> ---------------------------------------------------------------------
> To start a new topic, e-mail: [email protected]
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>
---------------------------------------------------------------------
To start a new topic, e-mail: [email protected]
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
