You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by Glenn Nielsen <gl...@voyager.apg.more.net> on 2002/05/07 13:51:23 UTC

Re: cvs commit: jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet JspServlet.java

Thanks for catching this Remmy.

Glenn

remm@apache.org wrote:
> 
> remm        02/05/06 20:15:53
> 
>   Modified:    jasper2/src/share/org/apache/jasper/servlet JspServlet.java
>   Log:
>   - Return a 404 instead of throwing an exception here (which in turn would cause
>     the response code to be 500).
> 
>   Revision  Changes    Path
>   1.12      +5 -4      jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java
> 
>   Index: JspServlet.java
>   ===================================================================
>   RCS file: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java,v
>   retrieving revision 1.11
>   retrieving revision 1.12
>   diff -u -r1.11 -r1.12
>   --- JspServlet.java   6 May 2002 04:33:16 -0000       1.11
>   +++ JspServlet.java   7 May 2002 03:15:53 -0000       1.12
>   @@ -1,7 +1,7 @@
>    /*
>   - * $Header: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java,v 1.11 2002/05/06 04:33:16 glenn Exp $
>   - * $Revision: 1.11 $
>   - * $Date: 2002/05/06 04:33:16 $
>   + * $Header: /home/cvs/jakarta-tomcat-jasper/jasper2/src/share/org/apache/jasper/servlet/JspServlet.java,v 1.12 2002/05/07 03:15:53 remm Exp $
>   + * $Revision: 1.12 $
>   + * $Date: 2002/05/07 03:15:53 $
>     *
>     * The Apache Software License, Version 1.1
>     *
>   @@ -272,7 +272,8 @@
>                // First check if the requested JSP page exists, to avoid
>                // creating unnecessary directories and files.
>                if (context.getResourceAsStream(jspUri) == null) {
>   -                throw new FileNotFoundException(jspUri);
>   +                response.sendError(HttpServletResponse.SC_NOT_FOUND, jspUri);
>   +                return;
>                }
>                boolean isErrorPage = exception != null;
>                synchronized(this) {
> 
> 
> 
> 
> --
> To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
> For additional commands, e-mail: <ma...@jakarta.apache.org>

-- 
----------------------------------------------------------------------
Glenn Nielsen             glenn@more.net | /* Spelin donut madder    |
MOREnet System Programming               |  * if iz ina coment.      |
Missouri Research and Education Network  |  */                       |
----------------------------------------------------------------------

--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>