KocsisDaniel/liferay-portal — pull request #111 by vilmospapp (2012-08-22 10:04:27+00:00)
Title: [TECHNICAL SUPPORT] LPS-29414 Added old style Document Library download link pattern to be ignored by GZipFilter

Hi Dani,

If the request of /c/document_library/get_file goes through GZipFilter Tomcat sets a timeout for 20000ms by default although we nowhere try to set a timeout in GZipFilter. Further more based on my conversation with Alex it's not really good to compress e.g.: JPEG files as they are already compressed and the new style URLs like /documents/* don't touch GZipFilter and they are really fast, we agreed that the removing of GZipFilter from the filter chain for the above mentioned URL could solve the problem. Actually the whole GetFileAction should be removed as it's not maintained recently as the new type downloads are used but I think it's quite complex and removing backward compatible functions should be done by core engineers who are much more familiar with the full concept of the features.

The used solution follows the patterns and instructs our own FilterMapping class to not pass the URLs to GZipFilter as the JAVA standard filter mapping cannot handle exclusions by patterns by default.

If you have any concerns, please don't hesitate to ask me.

Regards,
Vilmos