Hi Mike If you have your patch ready, go ahead and check it in. I still want to experiment with SSL via proxy before I commit mine.
By the way, as a native English speaker can you think of a better name for ExpectContinueMethod class? Cheers Oleg On Wed, 2003-03-12 at 17:05, Michael Becke wrote: > Looks reasonable to me. I am glad you added the proxy config to > TestLocalHostBase. I have done the same, but not checked it in yet. My > only suggestion would be to change the proxy system property names to > something like: > > httpclient.test.localProxyHost -> httpclient.test.proxy.host > httpclient.test.localProxyPort -> httpclient.test.proxy.port > > This is minor, but as with the test host the proxy doesn't have to be local. > > Mike > > [EMAIL PROTECTED] wrote: > > DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG > > RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT > > <http://nagoya.apache.org/bugzilla/show_bug.cgi?id=17821>. > > ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND > > INSERTED IN THE BUG DATABASE. > > > > http://nagoya.apache.org/bugzilla/show_bug.cgi?id=17821 > > > > 100-Continue Error > > > > > > > > > > > > ------- Additional Comments From [EMAIL PROTECTED] 2003-03-12 10:18 ------- > > 'Expect: 100-continue' related logic currently spread over a few classes has > > been refactored into an abstract class of its own. > > Folks, please give me some feedback. > > Oleg > > > > --------------------------------------------------------------------- > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
