httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From i...@apache.org
Subject cvs commit: httpd-2.0/docs/manual/mod mod_deflate.xml
Date Wed, 07 Aug 2002 15:26:17 GMT
ianh        2002/08/07 08:26:17

  Modified:    .        CHANGES
               modules/filters mod_deflate.c
               docs/manual/mod mod_deflate.xml
  Log:
  mod-deflate now checks to make sure gzip-only-text-html is set to 1, not anything.
  This will allow things like...
  SetEnv gzip-only-text/html 1
  BrowserMatch "GECKO" gzip-only-text/html=0
  
  Obtained from: André Schild <A.Schild@aarboard.ch>
  Reviewed by:   Ian Holsman
  
  Revision  Changes    Path
  1.881     +4 -0      httpd-2.0/CHANGES
  
  Index: CHANGES
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/CHANGES,v
  retrieving revision 1.880
  retrieving revision 1.881
  diff -u -r1.880 -r1.881
  --- CHANGES	6 Aug 2002 16:31:49 -0000	1.880
  +++ CHANGES	7 Aug 2002 15:26:15 -0000	1.881
  @@ -1,5 +1,9 @@
   Changes with Apache 2.0.40
   
  +  *) mod-defalte now checks to make sure that 'gzip-only-text/html' is
  +     set to 1, so we can exclude things from the general case with
  +     browsermatch. [Ian Holsman, Andre Schild <A.Schild@aarboard.ch>]
  +  
     *) Accept multiple leading /'s for requests within the DocumentRoot.
        PR 10946  [William Rowe, David Shane Holden <dpejesh@yahoo.com>]
   
  
  
  
  1.22      +7 -4      httpd-2.0/modules/filters/mod_deflate.c
  
  Index: mod_deflate.c
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/modules/filters/mod_deflate.c,v
  retrieving revision 1.21
  retrieving revision 1.22
  diff -u -r1.21 -r1.22
  --- mod_deflate.c	28 Jun 2002 08:40:24 -0000	1.21
  +++ mod_deflate.c	7 Aug 2002 15:26:17 -0000	1.22
  @@ -277,10 +277,13 @@
            * other than text/html, so set gzip-only-text/html
            * (with browsermatch) for them
            */
  -        if ((r->content_type == NULL
  -             || strncmp(r->content_type, "text/html", 9))
  -            && apr_table_get(r->subprocess_env, "gzip-only-text/html")) {
  -            ap_remove_output_filter(f);
  +        if (r->content_type == NULL
  +             || strncmp(r->content_type, "text/html", 9)) {
  +            const char *env_value = apr_table_get(r->subprocess_env,
  +                                                  "gzip-only-text/html");
  +            if ( env_value == NULL || strcmp(env_value,"1") ) {
  +                ap_remove_output_filter(f);
  +            }
               return ap_pass_brigade(f->next, bb);
           }
   
  
  
  
  1.7       +7 -3      httpd-2.0/docs/manual/mod/mod_deflate.xml
  
  Index: mod_deflate.xml
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/docs/manual/mod/mod_deflate.xml,v
  retrieving revision 1.6
  retrieving revision 1.7
  diff -u -r1.6 -r1.7
  --- mod_deflate.xml	2 Aug 2002 05:17:31 -0000	1.6
  +++ mod_deflate.xml	7 Aug 2002 15:26:17 -0000	1.7
  @@ -25,9 +25,13 @@
       <a href="../filter.html">filter</a>.  The following directive
       will enable compression for documents in the container where it
       is placed:</p>
  -    <p><strong>Most popular browsers can not handle compression of all content
  -        so you may want to enable the 'gzip-only-text/html' note (see below)
  -    </strong></p>
  +    <strong>
  +    <p>Most popular browsers can not handle compression of all content
  +        so you may want to set the 'gzip-only-text/html' note to 1 to only 
  +        allow html files to be compressed (see below)</p>
  +    <p>if you set this to anything but '1' it will be ignored, so you can do
  +       negative matches</p>
  +    </strong>
   
   <example>SetEnv gzip-only-text/html 1<br />
   SetOutputFilter DEFLATE
  
  
  

Mime
View raw message