Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 25114 - mod_xslt should not set Content-Encoding
Summary: mod_xslt should not set Content-Encoding
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Apache Team - Bugzilla Reports
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-07-23 05:06 UTC by Take Vos
Modified: 2004-04-23 06:59 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
removes the code that adds the Content-Encoding header. (mod_xslt-1.0.5a-noencoding.patch,754 bytes, patch)
2003-07-23 05:07 UTC, Take Vos
Details | Diff
This one is actualy tested :-) (mod_xslt-1.0.5a-noencoding.patch,3.53 KB, patch)
2003-07-23 06:14 UTC, Take Vos
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Take Vos 2003-07-23 05:06:21 UTC
mod_xslt sets the Content-Encoding http header to the charset in use. 
The charset should be encoded in the Content-Type (which it does correctly). 
 
The Content-Encoding should only be used for wrappers, such as compression methods, not 
charsets. This both Safari and Lynx don't understand this. 

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Take Vos 2003-07-23 05:07:22 UTC
Created attachment 14909 [details, diff]
removes the code that adds the Content-Encoding header.
Comment 2 Take Vos 2003-07-23 06:14:55 UTC
Created attachment 14916 [details, diff]
This one is actualy tested :-)
Comment 3 Take Vos 2003-07-23 06:15:41 UTC
Sorry, first patch was wrong. 
Took me some time to get the second patch correct. 
Comment 4 Donny Davies (RETIRED) gentoo-dev 2003-07-23 09:36:23 UTC
why dont you send your patches to the upsteam author instead.  ok?
Comment 5 Take Vos 2003-07-23 10:25:01 UTC
I have, I always send these patches to both ebuild maintainer and the 
developer. 
Comment 6 Donny Davies (RETIRED) gentoo-dev 2003-07-28 19:01:39 UTC
hmm you sent two patches...

you attached each more than once i think...

would you perhaps care to create a single patch, with all of your
latest fixes?  I just want to be sure you're settled on your fixes
and make it easier to process.

Any news from the author?
Comment 7 Chuck Short (RETIRED) gentoo-dev 2004-04-23 06:59:34 UTC
Patch has been added.