https://issues.apache.org/bugzilla/show_bug.cgi?id=55305
Mark Thomas <ma...@apache.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |WONTFIX
--- Comment #3 from Mark Thomas <ma...@apache.org> ---
The supplied patch has a bug - it fails to always set the content type for HEAD
requests.
After reviewing the code, I do have concerns with moving the setting of the
content type. The content type needs to be set to different values for range
requests and I'd rather see it set once correctly rather than multiple times
and be overwritten.
There is, however, a simple solution to the particualr problem that triggered
this report. The filter can use the same mechanism as the default servlet
(ServletContext.getMimeType()) to determine the mime type if the content type
is not set. That is not fool-proof but it is no worse than the change that was
proposed to the default servlet and it remains within the filter's control.
--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
