Mantis Bugtracker
Main | My View | View Issues | Change Log | Docs
  

Viewing Issue Simple Details [ Jump to Notes ] [ View Advanced ] [ Issue History ] [ Print ]
ID Category Severity Reproducibility Date Submitted Last Update
0000989 [Resin] minor always 03-13-06 14:32 04-13-06 10:59
Reporter ferg View Status public  
Assigned To ferg
Priority normal Resolution fixed  
Status closed   Product Version 3.0.18
Summary 0000989: JSP wrapper issue
Description (rep by Tony Field)

I am wrapping a ServletResponse, then am creating a request dispatcher and then including a standard JSP using my wrapped response:

My servlet uses a manager class to handle includes. Here is the interesting code:

    public ServletRRManager(HttpServlet servlet, HttpServletRequest request, HttpServletResponse response)
    {
        this.servlet = servlet;
        this.request = new MyHttpServletRequest(request);
        responseStack = new Stack<MyHttpServletResponse>();
        responseStack.push(new MyHttpServletResponse(response));
    }
    public void include(String path) throws IncludeException
    {
        RequestDispatcher rd = request.getRequestDispatcher(path);
        try
        {
            MyHttpServletResponse res = responseStack.peek();
            rd.include(request, res);
        }
        catch (Exception e)
        {
            throw new IncludeException("Failure including resource at " + path, e);
        }
    }

...

My wrapper class is really simple - basically it captures the output of the getWriter() content in a StringWriter and the getOutputStream() content in a ByteArrayOutputStream. My wrapper has accessor methods for this data and I process it before streaming it to the actual response (this processing is effectively the ultimate goal of the whole pile of code - we are refactoring our caching engine).

I have noticed that I can't include a JSP if I have already grabbed the outputStream because the JSP will always try to grab the writer. This seems reasonable.

However, if my caller grabs the writer instead, then includes a JSP, then tries to either write to the writer again or grab it again from the response wrapper, it fails because somewhere in the JSP include code the checkError() flag of the PrintWriter has been set: it seems that including a JSP "corrupts" my PrintWriter.
Additional Information
Attached Files

- Relationships

- Notes
(0000941)
anonymous
03-17-06 20:17

 Basically, in the com.caucho.jsp.Page.pageservice() method, after determining that the response is not a CauchoResponse it is wrapped then it executes the service() method in a try block. In the finally block the "response" is closed, and this closes the underlying stream and ultimately my printwriter.

You probably do want to close your wrapper but I don't think it makes sense to close the underlying writer, do you?.

Here is the stack trace that shows you where the close() call is happening that should not be happening:

java.lang.Exception: Stack trace
    at java.lang.Thread.dumpStack(Thread.java:1176)
    at com.fatwire.cs.reqres.servlet.CSHttpServletResponse$CSPrintWriter.close(CSHttpServletResponse.java:280)
    at com.caucho.server.connection.ToCharResponseAdapter$ToCharResponseStreamWrapper.close(ToCharResponseAdapter.java:142)
    at com.caucho.server.connection.ResponseAdapter.close(ResponseAdapter.java:308)
--> at com.caucho.jsp.Page.pageservice(Page.java:581)
    at com.caucho.server.dispatch.PageFilterChain.doFilter(PageFilterChain.java:159)
    at com.caucho.server.webapp.DispatchFilterChain.doFilter(DispatchFilterChain.java:115)
    at com.caucho.server.dispatch.ServletInvocation.service(ServletInvocation.java:229)
    at com.caucho.server.webapp.RequestDispatcherImpl.include(RequestDispatcherImpl.java:481)
    at com.caucho.server.webapp.RequestDispatcherImpl.include(RequestDispatcherImpl.java:346)
    at com.fatwire.cs.reqres.servlet.ServletRRManager.include(ServletRRManager.java:95)
    at com.fatwire.cs.reqres.servlet.TestServlet.doGet(TestServlet.java:39)
    at javax.servlet.http.HttpServlet.service(HttpServlet.java:115)
    at javax.servlet.http.HttpServlet.service(HttpServlet.java:92)
    at com.caucho.server.dispatch.ServletFilterChain.doFilter(ServletFilterChain.java:106)
    at com.caucho.server.webapp.WebAppFilterChain.doFilter(WebAppFilterChain.java:178)
    at com.caucho.server.dispatch.ServletInvocation.service(ServletInvocation.java:229)
    at com.caucho.server.http.HttpRequest.handleRequest(HttpRequest.java:268)
    at com.caucho.server.port.TcpConnection.run(TcpConnection.java:389)
    at com.caucho.util.ThreadPool.runTasks(ThreadPool.java:492)
    at com.caucho.util.ThreadPool.run(ThreadPool.java:425)
    at java.lang.Thread.run(Thread.java:613)
 
(0001090)
ferg
04-13-06 10:59

 server/172q
 

- Issue History
Date Modified Username Field Change
03-13-06 14:32 ferg New Issue
03-17-06 20:17 anonymous Note Added: 0000941
04-13-06 10:59 ferg Note Added: 0001090
04-13-06 10:59 ferg Assigned To  => ferg
04-13-06 10:59 ferg Status new => closed
04-13-06 10:59 ferg Resolution open => fixed
04-13-06 10:59 ferg Fixed in Version  => 3.0.19

Mantis 1.0.0rc3[^]
Copyright © 2000 - 2005 Mantis Group
30 total queries executed.
26 unique queries executed.
Powered by Mantis Bugtracker