hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 26070] - [RFE] Allow streaming of POST methods via chunked transfer encoding.
Date Mon, 08 Mar 2004 17:01:58 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=26070>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=26070

[RFE] Allow streaming of POST methods via chunked transfer encoding.





------- Additional Comments From olegk@apache.org  2004-03-08 17:01 -------
Mohammad,
The patch looks good to me. However, I'd like to see a few more things done
before the new implementation of the ChunkedOutputStream class can be committed. 

* Remove references to deprecated HttpConstants methods. Use corresponding
methods of the EncodingUtil class instead
* Provide at least rudimentary Javadocs.
* Parameterizable buffer size. Simple
ChunkedOutputStream#ChunkedOutputStream(OutputStream, int) constructor would do fine
* A SimpleHttpServer based test case would be just great. You may use
TestBadContentLength test unit as a reference implementation
* Use unified diff output format (diff -u) against CVS HEAD to produce the patch
if possible

If you are constrained in time, I can take over from here. However, I really
would like you to see this patch to its logical conclusion

Cheers,

Oleg

---------------------------------------------------------------------
To unsubscribe, e-mail: commons-httpclient-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-httpclient-dev-help@jakarta.apache.org


Mime
View raw message