Author: remy.maucherat(a)jboss.com
Date: 2009-11-06 09:48:00 -0500 (Fri, 06 Nov 2009)
New Revision: 1249
Modified:
trunk/java/org/apache/catalina/servlets/DefaultServlet.java
Log:
- Throw an exception when including a non existent resource.
Modified: trunk/java/org/apache/catalina/servlets/DefaultServlet.java
===================================================================
--- trunk/java/org/apache/catalina/servlets/DefaultServlet.java 2009-11-06 14:33:31 UTC
(rev 1248)
+++ trunk/java/org/apache/catalina/servlets/DefaultServlet.java 2009-11-06 14:48:00 UTC
(rev 1249)
@@ -24,6 +24,7 @@
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;
@@ -642,10 +643,9 @@
} 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(sm.getString("defaultServlet.missingResource",
- RequestUtil.filter(requestUri)));
+ // Therefore, throw an exception to notify the error.
+ throw new
FileNotFoundException(sm.getString("defaultServlet.missingResource",
+ RequestUtil.filter(requestUri)));
}
response.sendError(HttpServletResponse.SC_NOT_FOUND,
Show replies by date