Skip to content

Commit dd9ab16

Browse files
committed
Remove begin
1 parent d43d36f commit dd9ab16

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

lib/optimizely/cmab/cmab_client.rb

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -151,18 +151,16 @@ def _do_fetch_with_retry(url, request_body, retry_config, timeout)
151151
last_error = nil
152152

153153
(0..retry_config.max_retries).each do |attempt|
154-
begin
155-
return _do_fetch(url, request_body, timeout)
156-
rescue Optimizely::CmabInvalidResponseError
157-
# Do not retry, just re-raise
158-
raise
159-
rescue => e
160-
last_error = e
161-
if attempt < retry_config.max_retries
162-
@logger.info("Retrying CMAB request (attempt: #{attempt + 1} after #{backoff} seconds)...")
163-
sleep(backoff)
164-
backoff = [backoff * (retry_config.backoff_multiplier**(attempt + 1)), retry_config.max_backoff].min
165-
end
154+
return _do_fetch(url, request_body, timeout)
155+
rescue Optimizely::CmabInvalidResponseError
156+
# Do not retry, just re-raise
157+
raise
158+
rescue => e
159+
last_error = e
160+
if attempt < retry_config.max_retries
161+
@logger.info("Retrying CMAB request (attempt: #{attempt + 1} after #{backoff} seconds)...")
162+
sleep(backoff)
163+
backoff = [backoff * (retry_config.backoff_multiplier**(attempt + 1)), retry_config.max_backoff].min
166164
end
167165
end
168166

0 commit comments

Comments
 (0)