[Freeipa-devel] [PATCH] 253 Enable mod_deflate

[Petr Viktorin]

On 01/09/2013 02:06 PM, Petr Vobornik wrote:
> On 01/09/2013 11:59 AM, Petr Viktorin wrote:
>> 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.
>>
>
> Thanks. Compression for application/x-font-woff removed. Updated patch
> attached.
>

ACK


-- 
Petr³