Details
-
Improvement
-
Resolution: Fixed
-
Minor
-
1.1
-
None
-
-
Yes
-
Empty show more show less
-
Yes
Description
The FeedSyndicationServlet is currently hardcoded to set the contentType to "text/xml"
It would be useful to take advantage of the RSSAggregator for other kinds of feeds - such as an ICS feed based on events in the repository. ICS feeds require a content type of "text/calendar" not "text/xml"
The changes in the included patch are fairly minimal but allows a Feed to return a contentType that is then set in the servlet. The existing abstract servlet is updated to set a default contentType. Two unit tests were modified - one was updated to include the updated signature of the FeedSyndicationServlet.writeToResponse and the other test was modified to assert that the contentType of an RSS feed is set correctly to "text/xml" still after the changes.