Blanket disabling of compression for IE needs to be refined

sgodden's picture

Something that is starting to cause us problems now is the fact that compression of JS libraries is disabled for IE. The knowledge base article referenced indicates that this is a problem with IE 6 only, and can be fixed by installing the IE 6 service pack. We are happy to insist that our customers use IE versions that support compression, and the lack of compression on IE is making our bandwidth requirements far higher than they need be.

At the very least, I would like to see this as a switchable option, controlled by a "echo.disableiecompression" system property. I think the default should be that it is enabled.

Tod, what do you think?

sgodden's picture

I am going to provide an option to allow it

Tod, I am going to provide an option to allow it - so I won't change default behaviour for now.

Hopefully that is ok.

sgodden's picture

Committed in revision 2262

I have verified that default behaviour is still to disable compression, and that compression works ok when enabled.

sgodden's picture

I have just realised that I

I have just realised that I have committed a change with the wrong whitespace settings.

Apologies, I will sort this out quickly.

tliebeck's picture

Flag to enable is perfect

Flag to enable is perfect solution, people can try it out and see for themselves. It's quite obvious when it fails. :D

I'm not certain that that KB article describes all the failures though. For some reason I thought it could happen on IE7+ as well. It's been so long since I played with this. All of my data here is based on thinking back to a long time ago. Wasn't there something about firewalls/AV software disassembling things and then putting them back together incorrectly?

I do clearly recall the last time I tried to turn this feature on for IE though....a lot of people had problems with it.

All that said, if we can turn it on now, I'm down for it.

I dug up some old

I dug up some old posts:

http://echo.nextapp.com/site/node/2419
http://echo.nextapp.com/site/node/4676
http://echo.nextapp.com/site/node/1044#comment-2261
http://support.microsoft.com/?kbid=312496
http://support.microsoft.com/kb/837251

Both articles refer to IE6 only. Some quick googling also reveals issues when using IE5 or an unpatched IE6.

It would be nice if people that enable this flag post back results obtained in the field. I will definitely check this out.

Niels

golfman's picture

Where's the flah

What's the name of the flag and how do we enable it?

Step Ahead Software
Makers of Javelin Modeler+Coder for Java.
Automatic persistence of your object model using JDO and Hibernate.
http://stepaheadsoftware.com/products/javelin/javelin.htm

echo.allowiecompression

Specify "echo.allowiecompression" as a Java system property and set to true to enable compression for IE browsers.