From: Yui NARUSE <naruse@...>
Date: 2012-02-13T08:30:32+09:00
Subject: [ruby-core:42505] [ruby-trunk - Bug #6001] Retry idempotent HTTP requests for more errors


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/