You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2008/05/08 10:34:18 UTC
svn commit: r654446 -
/incubator/sling/trunk/sling/servlets-get/src/main/java/org/apache/sling/servlets/DefaultGetServlet.java
Author: fmeschbe
Date: Thu May 8 01:34:18 2008
New Revision: 654446
URL: http://svn.apache.org/viewvc?rev=654446&view=rev
Log:
Throw ResourceNotFoundException instead of sending 404 itself if the request
resource is a non existing resource
Modified:
incubator/sling/trunk/sling/servlets-get/src/main/java/org/apache/sling/servlets/DefaultGetServlet.java
Modified: incubator/sling/trunk/sling/servlets-get/src/main/java/org/apache/sling/servlets/DefaultGetServlet.java
URL: http://svn.apache.org/viewvc/incubator/sling/trunk/sling/servlets-get/src/main/java/org/apache/sling/servlets/DefaultGetServlet.java?rev=654446&r1=654445&r2=654446&view=diff
==============================================================================
--- incubator/sling/trunk/sling/servlets-get/src/main/java/org/apache/sling/servlets/DefaultGetServlet.java (original)
+++ incubator/sling/trunk/sling/servlets-get/src/main/java/org/apache/sling/servlets/DefaultGetServlet.java Thu May 8 01:34:18 2008
@@ -26,6 +26,7 @@
import org.apache.sling.api.SlingHttpServletRequest;
import org.apache.sling.api.SlingHttpServletResponse;
+import org.apache.sling.api.resource.ResourceNotFoundException;
import org.apache.sling.api.resource.ResourceUtil;
import org.apache.sling.api.servlets.SlingSafeMethodsServlet;
import org.apache.sling.servlets.helpers.HtmlRendererServlet;
@@ -76,6 +77,10 @@
streamerServlet = rendererMap.get(StreamRendererServlet.EXT_RES);
}
+ /**
+ * @throws ResourceNotFoundException if the resource of the request is a non
+ * existing resource.
+ */
@Override
protected void doGet(SlingHttpServletRequest request,
SlingHttpServletResponse response) throws ServletException,
@@ -83,9 +88,8 @@
// cannot handle the request for missing resources
if (ResourceUtil.isNonExistingResource(request.getResource())) {
- response.sendError(HttpServletResponse.SC_NOT_FOUND,
- "Resource not found at path " + request.getResource().getPath());
- return;
+ throw new ResourceNotFoundException(
+ request.getResource().getPath(), "No Resource found");
}
Servlet rendererServlet;