smartystreets_ruby_sdk 7.2.2 → 7.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b7f9ada984ffb0e3e24548b3be6bd6f78cbfad2fdd9779819f1621452eaf6363
-  data.tar.gz: de74b85c937140e0da205d110876eba409053e86537308b63ed2dac6744ea37e
+  metadata.gz: f3583fc90ecd6ea6b74e60c274560f4ccaca60ac3f87f0affb05d19875ea0db6
+  data.tar.gz: 5acee3177c57105e3057e417fcf53fe75b25f9a877e8329bef8d04808c132eec
 SHA512:
-  metadata.gz: c71926db5331e0680097fa46f8860d919dac69f679fe1815dbe33d1ae7d7674a061675f3549f32b9a9640998428e69aa817b00fdb2b303c49f3a2d5d17a7d6db
-  data.tar.gz: 95f228283349c90a727cd177f548e641f5b8df3c3c6163a766a16ba57a6ada57cb8db17caaf3b33d80be5acae0274dfce31a35f1fbf210009f754c1a5332d509
+  metadata.gz: 7f366e31a8ff91ecae194eac3d190f4c6875d295fcf6f7b1cc7551d31c7c930f3a4d670031c871d0fec1060f66bf194f61d3c14fe6f230ad60b23f1c1929d98d
+  data.tar.gz: ab15c7452caabe0a5493bb6dd30d1ad08690f6b5101dbf963bf7c2d685f32ca0cb6684aa47adbf26b9521102e8700af59e31a00a7a62a859cbbed6221436a6c0

data/lib/smartystreets_ruby_sdk/retry_sender.rb

@@ -1,8 +1,12 @@
+require 'net/http'
+require 'timeout'
+
 module SmartyStreets
   class RetrySender
     MAX_BACKOFF_DURATION = 10
     STATUS_TOO_MANY_REQUESTS = 429
     STATUS_TO_RETRY = [408, 429, 500, 502, 503, 504]
+    TIMEOUT_ERRORS = [Net::OpenTimeout, Net::ReadTimeout, Timeout::Error, Errno::ETIMEDOUT]
 
     def initialize(max_retries, inner, sleeper, logger)
       @max_retries = max_retries
@@ -16,29 +20,44 @@ module SmartyStreets
 
       (1..@max_retries).each do |i|
 
-        break if STATUS_TO_RETRY.include?(response.status_code.to_i) == false
-
-        if response.status_code.to_i == STATUS_TOO_MANY_REQUESTS
-          seconds_to_backoff = 10
-          if response.header.nil? == false
-            if Integer(response.header["Retry-After"], exception: false)
-              seconds_to_backoff = response.header["Retry-After"].to_i
+        if timeout_error?(response.error)
+          timeout_backoff(i, response.error)
+          response = @inner.send(request)
+        elsif !STATUS_TO_RETRY.include?(response.status_code.to_i)
+          break
+        else
+          if response.status_code.to_i == STATUS_TOO_MANY_REQUESTS
+            seconds_to_backoff = 10
+            unless response.header.nil?
+              if Integer(response.header["Retry-After"], exception: false)
+                seconds_to_backoff = response.header["Retry-After"].to_i
+              end
             end
+            rate_limit_backoff(seconds_to_backoff)
+          else
+            backoff(i)
           end
-          rate_limit_backoff(seconds_to_backoff)
-        else
-          backoff(i)
+          response = @inner.send(request)
         end
-        response = @inner.send(request)
       end
 
       response
     end
 
-    def backoff(attempt)
+    def timeout_error?(error)
+      return false if error.nil?
+
+      TIMEOUT_ERRORS.any? { |klass| error.is_a?(klass) }
+    end
+
+    def timeout_backoff(attempt, error)
+      backoff(attempt, "Timeout error (#{error.class}).")
+    end
+
+    def backoff(attempt, message = 'There was an error processing the request.')
       backoff_duration = [attempt, MAX_BACKOFF_DURATION].min
 
-      @logger.log("There was an error processing the request. Retrying in #{backoff_duration} seconds...")
+      @logger.log("#{message} Retrying in #{backoff_duration} seconds...")
       @sleeper.sleep(backoff_duration)
     end
 

data/lib/smartystreets_ruby_sdk/version.rb

@@ -1,3 +1,3 @@
 module SmartyStreets
-  VERSION = '7.2.2' # DO NOT EDIT (this is updated by a build job when a new release is published)
+  VERSION = '7.2.3' # DO NOT EDIT (this is updated by a build job when a new release is published)
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: smartystreets_ruby_sdk
 version: !ruby/object:Gem::Version
-  version: 7.2.2
+  version: 7.2.3
 platform: ruby
 authors:
 - SmartyStreets SDK Team