hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeffrey Dever <jsde...@sympatico.ca>
Subject Re: [PATCH] PostMethod & PutMethod revision (take 3)
Date Fri, 31 Jan 2003 22:19:15 GMT
The local and webapp tests are fine here (except that one timeout issue 
that has been around for a while).

There are a few things that could be tweaked, but please submit the 
patch so we can all benefit from it.  Very high quality, great work!

Jandalf


Kalnichevski, Oleg wrote:

>Bug fixes:
>
>http://nagoya.apache.org/bugzilla/show_bug.cgi?id=11095
>http://nagoya.apache.org/bugzilla/show_bug.cgi?id=11653
>http://nagoya.apache.org/bugzilla/show_bug.cgi?id=14731
>
>Changelog:
>
>- Abstract EntityEnclosingMethod class has been introduced to encapsulate common behaviour
of all entity enclosing methods 
>- "Expect: 100-continue" header support
>- HttpClient does not hang indefinitely if the server of origin does not return status
code 100 when expected. HttpClient resumes sending request body after 3 seconds if no response
is sent
>- Support for chunk encoded requests in all entity enclosing methods
>- Entity enclosing methods do not allow automatic redirection
>- More robust (or so I'd like to hope) request content buffering logic
>- PostMethod inherited from EntityEnclosingMethod class
>- PutMethod inherited from EntityEnclosingMethod class
>- Older methods of PostMethod dealing with url-encoded parameters deprecated in favour
of new setRequestBody(NameValuePair[]) method
>
>Feedback, critique appreciated, as always.
>
>Jeff, Mike
>I decided to not touch request body buffering as yet. The patch is getting really unwieldy.
I would like to have it committed, if you confirm it does not break test cases on your systems.
We can deal with smaller issues and improvements incrementally once the patch has been checked
in
>
>Cheers
>
>Oleg
>
>PS: The patch appears to have exceeded the allowed maximum. It has just been thrown back
to me ;-) 
>I am afraid I'll have to compress the file
>  
>
>------------------------------------------------------------------------
>
>---------------------------------------------------------------------
>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