[Freeipa-devel] [PATCH] 253 Enable mod_deflate

Petr Vobornik pvoborni at redhat.com
Thu Jan 17 16:23:59 UTC 2013


On 01/09/2013 05:18 PM, Petr Viktorin wrote:
> 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
>
>
Pushed to master

-- 
Petr Vobornik




More information about the Freeipa-devel mailing list