revert

Author: GrantBirki

lib/hooks/utils/retry.rb

@@ -9,24 +9,11 @@ module Retry
   # Should the number of retries be reached without success, the last exception will be raised
   #
   # @param log [Logger] The logger to use for retryable logging
-  # @raise [ArgumentError] If no logger is provided or configuration values are invalid
+  # @raise [ArgumentError] If no logger is provided
   # @return [void]
   def self.setup!(log: nil, log_retries: ENV.fetch("RETRY_LOG_RETRIES", "true") == "true")
     raise ArgumentError, "a logger must be provided" if log.nil?
 
-    # Security: Validate and bound retry configuration values
-    retry_sleep = ENV.fetch("DEFAULT_RETRY_SLEEP", "1").to_i
-    retry_tries = ENV.fetch("DEFAULT_RETRY_TRIES", "10").to_i
-
-    # Bounds checking to prevent resource exhaustion
-    if retry_sleep < 0 || retry_sleep > 300  # Max 5 minutes between retries
-      raise ArgumentError, "DEFAULT_RETRY_SLEEP must be between 0 and 300 seconds, got: #{retry_sleep}"
-    end
-
-    if retry_tries < 1 || retry_tries > 50   # Max 50 retries to prevent infinite loops
-      raise ArgumentError, "DEFAULT_RETRY_TRIES must be between 1 and 50, got: #{retry_tries}"
-    end
-
     log_method = lambda do |retries, exception|
       # :nocov:
       if log_retries
@@ -41,8 +28,8 @@ def self.setup!(log: nil, log_retries: ENV.fetch("RETRY_LOG_RETRIES", "true") ==
     Retryable.configure do |config|
       config.contexts[:default] = {
         on: [StandardError],
-        sleep: retry_sleep,
-        tries: retry_tries,
+        sleep: ENV.fetch("DEFAULT_RETRY_SLEEP", "1").to_i,
+        tries: ENV.fetch("DEFAULT_RETRY_TRIES", "10").to_i,
         log_method:
       }
     end