freno-client 0.8.2 → 0.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 808823b23d76b3a540d9d17d2c6e28b428fd3cb387c941f79c749779393ac462
-  data.tar.gz: cf58a8db9c3d7fa2bea92727b3336a62fded79081723ee5e13555a72e6d68c62
+  metadata.gz: 0e3f93dd3d1186c20522072cea3d306a19bf253dab7a4cd61fd2ecbf9a16fb83
+  data.tar.gz: 416f2d0ba15489ce73547836b13b046fafca1b6c357ab1a482c7cc0a13212720
 SHA512:
-  metadata.gz: 1fb13835bd9d2c9dd64c8bc3ae6e9931e0bd16a82cfa0ffbc4ed276505e453d4cf8a371cd66e6a0da19c2755e1174ce2812c55d0b12ce781b7a52f08add3f838
-  data.tar.gz: 540f2de04000141a1c636ab169e178a99a48dc19fe53557f7afba59dd2ddd2e2c407178e8a4f5e2daeb82b54871eac21f7d369854f1e9d7b87bbde1226289ffc
+  metadata.gz: e07318b40abf5ccb2a87af80498ce638ee2fd6fa9778e86806fd1a17905c0e3f06ae10a4eb4a0e851d22d106607d77e9a8a5d8fe766d084fe6ffca08da6ec39c
+  data.tar.gz: 9a902ee6324835e5758382b20dd7f4ce7e21bff6069cda917fa44b8b5b3aaa16b29bb75842f36309e92199c9d1e95a60660e6648a8ef71635851ddfb69fb537e

data/.rubocop.yml

@@ -42,3 +42,7 @@ Style/StringLiterals:
   EnforcedStyle: double_quotes
 Style/SymbolArray:
   Enabled: false
+Style/InfiniteLoop:
+  Enabled: false
+Style/Next:
+  Enabled: false

data/lib/freno/client/version.rb

@@ -2,6 +2,6 @@
 
 module Freno
   class Client
-    VERSION = "0.8.2"
+    VERSION = "0.8.3"
   end
 end

data/lib/freno/throttler.rb

@@ -166,7 +166,7 @@ module Freno
       instrument(:called, store_names: store_names)
       waited = 0
 
-      loop do
+      while true
         unless circuit_breaker.allow_request?
           instrument(:circuit_open, store_names: store_names, waited: waited)
           raise CircuitOpen
@@ -175,19 +175,21 @@ module Freno
         if all_stores_ok?(store_names, **options)
           instrument(:succeeded, store_names: store_names, waited: waited)
           circuit_breaker.success
-          return yield
+          break
         end
 
         wait
         waited += wait_seconds
         instrument(:waited, store_names: store_names, waited: waited, max: max_wait_seconds)
 
-        next unless waited > max_wait_seconds
-
-        instrument(:waited_too_long, store_names: store_names, waited: waited, max: max_wait_seconds)
-        circuit_breaker.failure
-        raise WaitedTooLong.new(waited_seconds: waited, max_wait_seconds: max_wait_seconds)
+        if waited > max_wait_seconds
+          instrument(:waited_too_long, store_names: store_names, waited: waited, max: max_wait_seconds)
+          circuit_breaker.failure
+          raise WaitedTooLong.new(waited_seconds: waited, max_wait_seconds: max_wait_seconds)
+        end
       end
+
+      yield
     end
 
     private

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: freno-client
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.3
 platform: ruby
 authors:
 - GitHub
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-10-20 00:00:00.000000000 Z
+date: 2024-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -78,7 +78,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
+rubygems_version: 3.4.19
 signing_key: 
 specification_version: 4
 summary: A library for interacting with Freno, the throttler service