google-api-ruby-client icon indicating copy to clipboard operation
google-api-ruby-client copied to clipboard

Add `Errno::ECONNRESET` to retriable network errors

Open mzuzek opened this issue 1 year ago • 3 comments

Would it make sense to also include Errno::ECONNRESET (broken network connections) in RETRIABLE_ERRORS ? (via Google::Apis::TransmissionError - same like ECONNREFUSED and ETIMEDOUT in https://github.com/googleapis/google-api-ruby-client/pull/10026)

I've hit that recently in context of GCS file downloads on Windows Server 2019 Datacenter 10.0.17763, Ruby 3.0.4p208 (2022-04-12 revision 3fa771dded) [x64-mingw32] and google-apis-core 0.9.3.

mzuzek avatar May 04 '24 10:05 mzuzek

@dazuma @bajajneha27 @NivedhaSenthil @SandeepTuniki

Would you see any reason why Errno::ECONNRESET should not be made retriable ?

mzuzek avatar May 14 '24 08:05 mzuzek

I think this makes sense, and is in line with the Storage error strategy currently being worked on internally. We should add it here: https://github.com/googleapis/google-api-ruby-client/blob/main/google-apis-core/lib/google/apis/core/http_command.rb#L295

dazuma avatar Jun 05 '24 19:06 dazuma

Thank you @dazuma for your feedback on this issue.

mzuzek avatar Jun 10 '24 07:06 mzuzek