[cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread Anthony Green
I found this while debugging Azureus. We automatically decompress compressed HTTP responses, but still report them as having compressed content. This patch removes the Content-Encoding header in these cases. This appears to be what Sun does in these cases. Ok? tromey: I would also like to

Re: [cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread Tom Tromey
Anthony == Anthony Green [EMAIL PROTECTED] writes: Anthony I just want to get this patch into rawhide. My feeling is that it Anthony solves an incompatibility, and introduces no regressions. I agree, it is the minimal required improvement. Please check it in. I think what we want to do is

Re: [cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread Anthony Green
On Mon, 2006-01-16 at 10:52 -0700, Tom Tromey wrote: When I run that case program, if I specify that it should ask for a gzip encoding, Sun's protocol handler hands back a gzipped stream -- i.e., it does not uncompress. I didn't even notice this option! I think either behavior is correct,

Re: [cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread Tom Tromey
David == David Daney [EMAIL PROTECTED] writes: Also I wonder about throwing an exception if we don't recognize the content-encoding. It seems to me that the caller might well recognize it somehow. David How would you make this behavior compatible with java.net.*? David What exception would

Re: [cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread Tom Tromey
Anthony == Anthony Green [EMAIL PROTECTED] writes: Anthony I found this while debugging Azureus. We automatically decompress Anthony compressed HTTP responses, but still report them as having compressed Anthony content. This patch removes the Content-Encoding header in these Anthony cases.

Re: [cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread Anthony Green
On Mon, 2006-01-16 at 12:49 -0700, Tom Tromey wrote: Anthony == Anthony Green [EMAIL PROTECTED] writes: Anthony I just want to get this patch into rawhide. My feeling is that it Anthony solves an incompatibility, and introduces no regressions. I agree, it is the minimal required

Re: [cp-patches] Patch: fix Content-Encoding for compressed HTTP responses

2006-01-16 Thread David Daney
Tom Tromey wrote: Anthony == Anthony Green [EMAIL PROTECTED] writes: Anthony I found this while debugging Azureus. We automatically decompress Anthony compressed HTTP responses, but still report them as having compressed Anthony content. This patch removes the Content-Encoding header in