[PATCH][HttpClient] Bug 12607 fix w/ test + typo fixes

["Elwin, Martin" <me...@rsasecurity.com>]

ChunkedInputStream.java.diff is a patch for the 12607 bug. The solution was
proposed by the bug submitter and I've verified it as well.

Simple real-world test to verify the bug and the fix:

.../jakarta-commons/httpclient/target/examples$ java -classpath
.;../classes;../../lib/commons-logging.jar TrivialApp http://slashdot.org

TestStreams.java is updated to include a test of this as well (and a small
fix to the suite() method so that the test is executed at all! :). Splitting
the testChunkedInputStream into several methods (or tests) might be good.

The two other diffs are for a typo in a trace output in HttpConnection and a
typo in the syntax description of TrivialApp.

Kindly,

/M

------------
Martin Elwin
@N59.29606 E18.08052 (WGS-84)

Re: [PATCH][HttpClient] Bug 12607 fix w/ test + typo fixes

[Ryan Hoegg <rh...@isisnetworks.net>]

I have tested the patch to ChunkedInputStream on my local copy and it 
works; in fact, it solved a larger problem I was having integrating 
HttpClient into the Apache XML-RPC project.

Thanks Martin!

--
Ryan Hoegg
ISIS Networks

Elwin, Martin wrote:

>ChunkedInputStream.java.diff is a patch for the 12607 bug. The solution was
>proposed by the bug submitter and I've verified it as well.
>
>Simple real-world test to verify the bug and the fix:
>
>.../jakarta-commons/httpclient/target/examples$ java -classpath
>.;../classes;../../lib/commons-logging.jar TrivialApp http://slashdot.org
>
>TestStreams.java is updated to include a test of this as well (and a small
>fix to the suite() method so that the test is executed at all! :). Splitting
>the testChunkedInputStream into several methods (or tests) might be good.
>
>The two other diffs are for a typo in a trace output in HttpConnection and a
>typo in the syntax description of TrivialApp.
>
>Kindly,
>
>/M
>
>------------
>Martin Elwin
>@N59.29606 E18.08052 (WGS-84)
>


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