When enabling compression send 'gzip' as Content-Encoding instead of the

deprecated 'x-gzip'. Acording to RFC2616 clients should handle "x-gzip"
but it seems that at least some Android devices does not.

In general the webserver instead of the CGI should probably do
compression, but this is an easy fix.

PR:		www/152202
Reported by:	Tom Judge <tom@tomjudge.com>
This commit is contained in:
Simon L. B. Nielsen 2010-11-13 16:37:18 +00:00
parent 918b4beaf1
commit 8fb3387486
Notes: svn2git 2020-12-08 03:00:23 +00:00
svn path=/www/; revision=36586

View file

@ -45,8 +45,8 @@
# OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
# SUCH DAMAGE.
#
# $FreeBSD: www/en/cgi/cvsweb.cgi,v 1.91 2007/03/17 22:28:31 simon Exp $
# $Id: cvsweb.cgi,v 1.92 2008-09-23 16:31:45 wosch Exp $
# $FreeBSD: www/en/cgi/cvsweb.cgi,v 1.92 2008/09/23 16:31:45 wosch Exp $
# $Id: cvsweb.cgi,v 1.93 2010-11-13 16:37:18 simon Exp $
# $Idaemons: /home/cvs/cvsweb/cvsweb.cgi,v 1.84 2001/10/07 20:50:10 knu Exp $
#
###
@ -142,7 +142,7 @@ EOM
$CheckoutMagic = '~checkout~';
$cgi_style::hsty_base = 'http://www.FreeBSD.org';
$_ = q$FreeBSD: www/en/cgi/cvsweb.cgi,v 1.91 2007/03/17 22:28:31 simon Exp $;
$_ = q$FreeBSD: www/en/cgi/cvsweb.cgi,v 1.92 2008/09/23 16:31:45 wosch Exp $;
@_ = split;
$cgi_style::hsty_date = "@_[3,4]";
@ -4281,7 +4281,7 @@ sub http_header(;$$)
|| (defined($CMD{gzip}) && open(GZIP, "| $CMD{gzip} -1 -c")))
{
push(@headers, 'Content-Encoding: x-gzip');
push(@headers, 'Content-Encoding: gzip');
push(@headers, 'Vary: Accept-Encoding'); # RFC 2616, 14.44
print join("\r\n", @headers) . "\r\n\r\n";