bugfix

lib/network_resiliency/adapter/http.rb

@@ -50,6 +50,9 @@ def with_resilience(action, destination, idempotent, &block)
             timeouts = timeouts.last(1)
           end
 
+          original_max_retries = self.max_retries
+          self.max_retries = 0 # disable
+
           attempts = 0
           ts = -NetworkResiliency.timestamp
 
@@ -73,6 +76,7 @@ def with_resilience(action, destination, idempotent, &block)
           ensure
             ts += NetworkResiliency.timestamp
             set_timeout.call(original_timeout)
+            self.max_retries = original_max_retries
 
             NetworkResiliency.record(
               adapter: "http",
@@ -107,12 +111,7 @@ def transport_request(req, &block)
 
           idepotent = Net::HTTP::IDEMPOTENT_METHODS_.include?(req.method)
 
-          retries = self.max_retries
-          self.max_retries = 0 # disable
-
           with_resilience(:request, destination, idepotent) { super }
-        ensure
-          self.max_retries = retries
         end
       end
     end

spec/http_spec.rb

@@ -274,6 +274,13 @@
         end
       end
     end
+
+    context "when NetworkResiliency is disabled" do
+      before { NetworkResiliency.disable! }
+
+      it { is_expected.to eq "OK" }
+      it { expect(NetworkResiliency).not_to have_received(:record) }
+    end
   end
 
   describe "normalize path" do

spec/redis_spec.rb

@@ -253,6 +253,7 @@
     context "when NetworkResiliency is disabled" do
       before { NetworkResiliency.disable! }
 
+      it { expect(redis.ping).to eq "PONG" }
       it { is_expected.not_to have_received(:record) }
     end