PHP 4.0 Bug #10070 Updated: incorrect ob_gzhandler implementation

ID: 10070 User Update by: mnot@akamai.com Old-Status: Feedback Status: Open Bug Type: Zlib Related Description: incorrect ob_gzhandler implementation IIRC, it advertises both, so it shouldn't be an issue. I don't have a windows box around to make sure with... Previous Comments: --------------------------------------------------------------------------- [2001-03-30 19:45:38] avsm@php.net Just to clarify a bit more, and see if a workaround is worthwhile: Currently, if both gzip and deflate are present in the Accept-Encoding, we will prefer the gzip. The only good reason for a workaround in the code is if a widespread browser presents deflate only, and then proceeds to break. Do these broken versions of IE present Accept-Encoding: deflate _only_ ? And which versions are they? Anil --------------------------------------------------------------------------- [2001-03-30 14:59:49] sniper@php.net If a browser breaks RFCs -> not our problem. Closed. --Jani --------------------------------------------------------------------------- [2001-03-30 13:07:42] avsm@php.net Applied a patch to add Vary: headers if we add a Content-Encoding: header. Regarding the browser compatibility with Deflate, I think we should just leave it like it is. _They_ break the RFC; encourage people to upgrade to modern, non-broken browsers instead ? --------------------------------------------------------------------------- [2001-03-29 21:21:53] mnot@akamai.com ob_gzhandler doesn't appear to set a Vary: header on responses, which means that proxy caches could store a compressed version of the resource and serve it to clients which can't handle the encoding. This is very difficult to debug for Web sites, and should be fixed ASAP. For an example of the proper headers to send, see the PHP at: http://www.mnot.net/cgi_buffer/ Also, in my experience, some versions of MSIE don't handle deflate compession correctly, even though their Accept-Encoding claims they do. It might be better to avoid deflate altogether, as gzip support is pretty much the unwritten standard. Cheers, --------------------------------------------------------------------------- Full Bug description available at: http://bugs.php.net/?id=10070