[Freeipa-devel] [PATCH] 253 Enable mod_deflate

[Petr Vobornik]

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.
-- 
Petr Vobornik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-pvoborni-0253-1-Enable-mod_deflate.patch
Type: text/x-patch
Size: 1793 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130109/91273871/attachment.bin>