Re: Code Review 6993490: SocketTimeoutException on HTTP keep-alive connections

2010-10-21 Thread Michael McMahon
Chris Hegarty wrote: Hi Michael, The resetting of the read timeout after, error stream buffering, Expect 100-Continue handling, and keepalive cleaner, may be problematic if the read timeout is the default value. The default read tmeout is -1 ( not set ), but Networkclient.setReadTimeout ignor

Code Review 6993490: SocketTimeoutException on HTTP keep-alive connections

2010-10-21 Thread Chris Hegarty
Hi Michael, The resetting of the read timeout after, error stream buffering, Expect 100-Continue handling, and keepalive cleaner, may be problematic if the read timeout is the default value. The default read tmeout is -1 ( not set ), but Networkclient.setReadTimeout ignores values less than 0.