Issue #6001 has been updated by Yui NARUSE.


I agree the general direction to add more exceptions like ECONNABORTED.
But I doubt Timeout::Error and should be added.

A general principle is to rescue temporally errors.
I don't think Errno::EINVAL, OpenSSL::SSL::SSLError are such errors.

# anyway, the patch includes Errno::ECONNRESET twice.
----------------------------------------
Bug #6001: Retry idempotent HTTP requests for more errors
https://bugs.ruby-lang.org/issues/6001

Author: Eric Hodel
Status: Open
Priority: Normal
Assignee: 
Category: lib
Target version: 2.0.0
ruby -v: ruby 2.0.0dev (2012-02-11 trunk 34547) [x86_64-darwin11.2.0]


The net-http-persistent gem implements #5790 / #5813 but for more error types:

https://github.com/drbrain/net-http-persistent/blob/1b5f84d020/lib/net/http/persistent.rb#L811-813

These additional error types should be rescued by net/http and retried.


-- 
http://bugs.ruby-lang.org/