Code Review 6993490: SocketTimeoutException on HTTP keep-alive connections
michael.x.mcmahon at oracle.com
Thu Oct 21 08:12:53 PDT 2010
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 ignores
> values less than 0. So if no timeout is explicitly set, -1, any of
> these three pieces of code may not actually correctly reset the
> timeout to infinity. This may lead to SocketTimeoutException on
> persistent connections returned from the cache.
> The most problematic case is error stream buffering as it sets a
> timeout of 60 millis.
More information about the net-dev