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 [EMAIL PROTECTED]  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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to