You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by "Paul C. Bryan" <pb...@creatia.com> on 2001/12/02 06:28:53 UTC

[HttpClient] OOPS [PATCH] Bug 5018

To my surprise, I just discovered that ResponseInputStream is already 
attempting to support chunked transfer encoding. Therefore, the patch I 
previously submitted to address this bug is moot. Please disregard my 
previous patch postings for bug 5018.

However, I have identified the bug in ResponseInputStream. At line 99, 
it attempts to find the index of the constant "chunked" in a 
case-sensitive manner. Some servers, such as WebLogic, reply with 
"Chunked", thus causing this test to fail. The attached patch should 
solve this problem.

My apologies to those who have invested any time in reviewing my 
previous patch. Chalk it up to someone who is not initimately familiar 
with the code (yet).

Yours truly,

Paul C. Bryan <pb...@creatia.com>
http://pbryan.net/

Re: [HttpClient] OOPS [PATCH] Bug 5018

Posted by Remy Maucherat <re...@apache.org>.
> To my surprise, I just discovered that ResponseInputStream is already 
> attempting to support chunked transfer encoding. Therefore, the patch I 
> previously submitted to address this bug is moot. Please disregard my 
> previous patch postings for bug 5018.
> 
> However, I have identified the bug in ResponseInputStream. At line 99, 
> it attempts to find the index of the constant "chunked" in a 
> case-sensitive manner. Some servers, such as WebLogic, reply with 
> "Chunked", thus causing this test to fail. The attached patch should 
> solve this problem.

Yes, that's correct. The value should be case-insensitive.

Remy


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>