DO NOT REPLY [Bug 16113] - removing then replacing a jsp page continues to give a 404

[bugzilla]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16113>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=16113

removing then replacing a jsp page continues to give a 404





------- Additional Comments From [EMAIL PROTECTED]  2004-03-05 13:26 -------
Simply compare JspRuntimeContext.checkCompile(), this calls 
ctxt.incrementRemoved() if a FileNotFoundException occur. JspServletWrapper 
only set a 404 if a FileNotFoundException occur and finish. 
 
Code from JspRuntineContext: 
            synchronized(jsw) { 
                try { 
                    ctxt.compile(); 
                } catch (FileNotFoundException ex) { 
                    ctxt.incrementRemoved(); 
                } catch (Throwable t) { 
                    jsw.getServletContext().log("Background compile failed", 
                                                t); 
                } 
            } 
 
Code from JspServletWrapper: 
 
       } catch (FileNotFoundException ex) { 
            String includeRequestUri = (String) 
                request.getAttribute("javax.servlet.include.request_uri"); 
            if (includeRequestUri != null) { 
                // This file was included. Throw an exception as 
                // a response.sendError() will be ignored by the 
                // servlet engine. 
                throw new ServletException(ex); 
            } else { 
                try { 
                    response.sendError(HttpServletResponse.SC_NOT_FOUND, 
                                      ex.getMessage()); 
                } catch (IllegalStateException ise) { 
                    log.error(Localizer.getMessage("jsp.error.file.not.found", 
                                                   ex.getMessage()), 
                              ex); 
                } 
 
 
add a simple ctxt.incrementRemoved in catch block and test case works.

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]