[Freeipa-devel] [PATCH] 253 Enable mod_deflate

[Petr Viktorin]

On 01/09/2013 08:41 AM, Martin Kosek wrote:
> On 01/08/2013 06:05 PM, Petr Vobornik wrote:
>> On 01/08/2013 05:58 PM, Rob Crittenden wrote:
>>> Petr Vobornik wrote:
>>>> Design page: http://www.freeipa.org/page/V3/WebUI_gzip_compression
>>>>
>>>> Enabled mod_deflate for:
>>>> * text/html (HTML files)
>>>> * text/plain (for future use)
>>>> * text/css (CSS files)
>>>> * text/xml (XML RPC)
>>>> * application/javascript (JavaScript files)
>>>> * application/json (JSON RPC)
>>>> * application/x-font-woff (woff fonts)
>>>>
>>>> Added proper mime type for woff fonts.
>>>> Disabled etag header because it doesn't work with mod_deflate.
>>>>
>>>> https://fedorahosted.org/freeipa/ticket/3326
>>>
>>> Should this be enabled on upgrades as well?
>>
>> Yes, I don't see a reason not to.
>
> This should be enabled on upgrades as is, since Petr bumped VERSION in
> install/conf/ipa.conf.
>
> We should carefully check that enabling it also for xmlrpc/json does not cause
> any grief.
>
> Martin
>

HTTP libraries won't ask for gzip if they can't handle it, so there 
shouldn't be any grief.
I tested the UI, installing client & replica, and the CLI tool. All work 
fine.

Just one thing: WOFF is already compressed so we shouldn't gzip it again.


-- 
Petr³