Author: markt Date: Mon Mar 22 21:38:52 2010 New Revision: 926355 URL: http://svn.apache.org/viewvc?rev=926355&view=rev Log: SRV.9.3 says we must throw a FNFE
Modified: tomcat/trunk/java/org/apache/catalina/servlets/DefaultServlet.java Modified: tomcat/trunk/java/org/apache/catalina/servlets/DefaultServlet.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/servlets/DefaultServlet.java?rev=926355&r1=926354&r2=926355&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/catalina/servlets/DefaultServlet.java (original) +++ tomcat/trunk/java/org/apache/catalina/servlets/DefaultServlet.java Mon Mar 22 21:38:52 2010 @@ -24,6 +24,7 @@ import java.io.ByteArrayInputStream; import java.io.ByteArrayOutputStream; import java.io.File; import java.io.FileInputStream; +import java.io.FileNotFoundException; import java.io.IOException; import java.io.InputStream; import java.io.InputStreamReader; @@ -703,13 +704,11 @@ public class DefaultServlet if (requestUri == null) { requestUri = request.getRequestURI(); } else { - // We're included, and the response.sendError() below is going - // to be ignored by the resource that is including us. - // Therefore, the only way we can let the including resource - // know is by including warning message in response - response.getWriter().write(RequestUtil.filter( - sm.getString("defaultServlet.missingResource", - requestUri))); + // We're included + // SRV.9.3 says we must throw a FNFE + throw new FileNotFoundException( + sm.getString("defaultServlet.missingResource", + requestUri)); } response.sendError(HttpServletResponse.SC_NOT_FOUND, --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org