Return-Path: 
 <dev-return-35109-apmail-httpd-dev-archive=httpd.apache.org@httpd.apache.org>
Delivered-To: apmail-httpd-dev-archive@httpd.apache.org
Received: (qmail 86287 invoked by uid 500); 9 Dec 2002 13:24:00 -0000
Mailing-List: contact dev-help@httpd.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@httpd.apache.org
list-help: <mailto:dev-help@httpd.apache.org>
list-unsubscribe: <mailto:dev-unsubscribe@httpd.apache.org>
list-post: <mailto:dev@httpd.apache.org>
Delivered-To: mailing list dev@httpd.apache.org
Received: (qmail 86269 invoked from network); 9 Dec 2002 13:24:00 -0000
Message-ID: <3DF4999F.7010801@apache.org>
Date: Mon, 09 Dec 2002 14:24:47 +0100
From: Henri Gomez <hgomez@apache.org>
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; fr-FR;
 rv:1.2) Gecko/20021126
X-Accept-Language: en-us, en, fr
MIME-Version: 1.0
To: dev@httpd.apache.org
Subject: Re: mod_deflate with mod_jk
References: <3DEE209A.7060004@apache.org>
 <Pine.WNT.4.50.0212041039140.1468-100000@Poste3947>
 <Pine.WNT.4.50.0212041039140.1468-100000@Poste3947>
 <3DF46CD3.1020003@apache.org> <n2m-g.1ec4u77sfw0a0ndparker@news.perlig.de>
 <3DF489E4.6030003@apache.org>
In-Reply-To: <3DF489E4.6030003@apache.org>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit
X-Spam-Rating: localhost.apache.org 1.6.2 0/1000/N
X-Spam-Rating: daedalus.apache.org 1.6.2 0/1000/N

Henri Gomez wrote:
> Andr� Malo wrote:
> 
>> * Henri Gomez wrote:
>>
>>
>>> Ok, it works with SetOutputFilter DEFLATE, I'm now wonder
>>> how to remove *.gif/*.jpg and *.js from being compressed.
>>
>>
>>
>> hmm.
>> <Files ~ \.(gif|jpeg|js)$>
>>   SetEnv no-gzip
>> </Files>
> 
> 
> Hum, it didn't seems to works on my iSeries.
> Neither with :
> 
> SetEnvIfNoCase Request_URI \.(?:gif|jpe?g|png)$ no-gzip dont-vary
> 
> 
>> ?
>> If mod_jk uses the appropriate Apache API call (ap_set_content_type), 
> 
> 
> Not yet, since we'll have to reparse the tomcat reply which is not
> done today.
> 
>> AddOutputFilterByType DEFLATE text/html
> 
> 
> I'll try to see how to add it...

The actual jk code in response processing is :

             if(!strcasecmp(header_names[h], "Content-type")) {
                 char *tmp = apr_pstrdup(r->pool, header_values[h]);
                 ap_content_type_tolower(tmp);
                 r->content_type = tmp;
	    }

I update to use ap_set_content_type(r, tmp) instead of r->content_type = 
tmp and it works now !!!

Regards.

BTW, the fix will be in JK 1.2.2 / 2.0.3, also in mod_webapp and
right now in jakarta-tomcat-connectors CVS...