Close connection in case of error.

[ixti]

Right now we close connection in client upon case of error only if client is persistent.
https://github.com/httprb/http/blob/master/lib/http/client.rb#L78

IMO that line should be just close without the guard.

@zanker @tarcieri thoughts?

[zanker]

I have no idea why I made it only close persistent ones. 👍 on fixing.

Sent from my iPhone

On Dec 12, 2015, at 06:02, Alexey Zapparov notifications@github.com wrote:

Right now we close connection in client upon case of error only if client is persistent.
https://github.com/httprb/http/blob/master/lib/http/client.rb#L78

IMO that line should be just close without the guard.

@zanker @tarcieri thoughts?

—
Reply to this email directly or view it on GitHub.

[sferik]

👍

[tarcieri]

also 👍