Browse Source

ResourceHttpRequestHandler sends content without content-type header if no media type found (SPR-7713)

3.0.x
Juergen Hoeller 14 years ago
parent
commit
f8402e9f82
  1. 11
      org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/resource/ResourceHttpRequestHandler.java

11
org.springframework.web.servlet/src/main/java/org/springframework/web/servlet/resource/ResourceHttpRequestHandler.java

@ -109,15 +109,13 @@ public class ResourceHttpRequestHandler extends WebContentGenerator implements H @@ -109,15 +109,13 @@ public class ResourceHttpRequestHandler extends WebContentGenerator implements H
MediaType mediaType = getMediaType(resource);
if (mediaType != null) {
if (logger.isDebugEnabled()) {
logger.debug("Determined media type [" + mediaType + "] for " + resource);
logger.debug("Determined media type '" + mediaType + "' for " + resource);
}
}
else {
if (logger.isDebugEnabled()) {
logger.debug("No media type found for " + resource + " - returning 404");
logger.debug("No media type found for " + resource + " - not sending a content-type header");
}
response.sendError(HttpServletResponse.SC_NOT_FOUND);
return;
}
// header phase
@ -207,7 +205,10 @@ public class ResourceHttpRequestHandler extends WebContentGenerator implements H @@ -207,7 +205,10 @@ public class ResourceHttpRequestHandler extends WebContentGenerator implements H
throw new IOException("Resource content too long (beyond Integer.MAX_VALUE): " + resource);
}
response.setContentLength((int) length);
response.setContentType(mediaType.toString());
if (mediaType != null) {
response.setContentType(mediaType.toString());
}
}
/**

Loading…
Cancel
Save