Fix an issue in the retry policy

Azure · Oct 31, 2016 · 771481b · 771481b
1 parent 2231cfa
commit 771481b
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -1,3 +1,8 @@
+2016.10 - version 0.11.3-preview
+
+ALL
+* Fixed an issue in retry policies.
+
 2016.10 - version 0.11.2-preview
 
 ALL

diff --git a/lib/azure/storage/core/filter/retry_filter.rb b/lib/azure/storage/core/filter/retry_filter.rb
@@ -136,8 +136,11 @@ def should_retry_on_error?(response, retry_data)
         end
       end
 
+      if (retry_data[:status_code] < 400)
+        retry_data[:retryable] = false;
+        return false;
       # Non-timeout Cases
-      if (retry_data[:status_code] >= 300 && retry_data[:status_code] != 408)
+      elsif (retry_data[:status_code] != 408)
         # Always no retry on "not implemented" and "version not supported"
         if (retry_data[:status_code] == 501 || retry_data[:status_code] == 505)
           retry_data[:retryable] = false;

diff --git a/lib/azure/storage/version.rb b/lib/azure/storage/version.rb
@@ -28,7 +28,7 @@ class Version
       # Fields represent the parts defined in http://semver.org/
       MAJOR = 0 unless defined? MAJOR
       MINOR = 11 unless defined? MINOR
-      UPDATE = 2 unless defined? UPDATE
+      UPDATE = 3 unless defined? UPDATE
       PRE = 'preview' unless defined? PRE
 
       class << self