Don't use infinity as default timeout

[RJ]

There is a default 'infinity' recv_timeout in hackney:request

infinity makes a poor default, imo.

[edgurgel]

Interesting that I got a similar comment on HTTPoison edgurgel/httpoison#64

[benoitc]

It's now set to 5s :) Not sure right now what is the best value for it.

[benoitc]

Also in the next version it shouldn't freeze anymore the pool.

[RJ]

@benoitc great :)
did you make another change related to the pool freezing, or are you just referring to the default timeout?

for the record, we started seeing connections (to s3) hanging indefinitely after upgrading from 0.13.0 to 1.1.0

[benoitc]

Pool locking will be handled in the next release. Basically there won't be
any locking or freezing waiting for a connection to be released. Instead we
will only make sure to maintain a max of reused connection with a maximum
time given to a connection to be kept alive. This version will be available
sometimes this week.

On Thu, Jul 30, 2015 at 1:27 PM Richard Jones notifications@github.com
wrote:

@benoitc https://github.com/benoitc great :)
did you make another change related to the pool freezing, or are you just
referring to the default timeout?

for the record, we started seeing connections (to s3) hanging indefinitely
after upgrading from 0.13.0 to 1.1.0

—
Reply to this email directly or view it on GitHub
#219 (comment).