code-ruby 5.2.4 → 6.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b2db565dfb611ea14df67af826fd2a7d425c4994a355fb2dea00f768263af06a
-  data.tar.gz: e227c3acd9a1f6bdbc1f60f1a3229266d1d48b3a850e3a9066275c9613057ad5
+  metadata.gz: 1aec9dbb3d232f6cd9f80c2094505b60ad1f6d3ffb974fcc99ce1ccf06d7772c
+  data.tar.gz: 706e9983a36f38bad9687abb1edbbab6a546e63e643397b7bcbd50a8e21549a7
 SHA512:
-  metadata.gz: dc5764588753a879e2fcc926e16e8a86895964192f15a692c941fcdc8fa6dfd578ebcd1eff3214d91ca82bc7dbaab0687866d3778bcdb5bd65fa37f8bdb2e190
-  data.tar.gz: 190d3f87a380bd783ba276f1b82935f740f0d5b3be52d3d44c5a6f728e7e95f0448df8a1f37734a5ea71c1f2ff0f3cc33e373155e3f6eaac6fafd770cf3ee07d
+  metadata.gz: 65f90487ee1e6110b87605d7014963b44664912591291c2c5d3bac4b72303bec0c7955825c2b0d9f50ec3c1d25bb63d5b0815732ad79c1a8d65505efc74b3414
+  data.tar.gz: 9715125c292a852fe2f18179c3305e5b1f890ffd20661c4db509d359dee2099a42930a4befddce72666e9825239c174b76b6e43789752dc7afa32f42f2ce8a77

data/VERSION

@@ -1 +1 @@
-5.2.4
+6.0.0

data/lib/code/concerns/shared.rb

@@ -319,10 +319,14 @@ class Code
 
           code_context.code_fetch(self)
         else
-          raise(
-            Error,
-            "#{code_operator.inspect} not defined on #{code_inspect}:#{code_name}"
-          )
+          if code_operator.to_s.match?(/\A[A-Z]/)
+            raise(
+              Error,
+              "#{code_operator.inspect} not defined on #{code_inspect}:#{code_name}"
+            )
+          end
+
+          Object::Nothing.new
         end
       end
 

data/lib/code/object/http.rb

@@ -199,6 +199,20 @@ class Code
       }.freeze
       DEFAULT_TIMEOUT = 30
       DEFAULT_OPEN_TIMEOUT = 10
+      DEFAULT_REDIRECTS = 0
+      DEFAULT_MAX_RETRIES = 0
+      DEFAULT_RETRY_DELAYS = [].freeze
+      MAX_REDIRECTS = 10
+      MAX_RETRIES = 5
+      MAX_RETRY_DELAY = 60
+      MAX_RESPONSE_BYTES = 50.megabytes
+      SAFE_RETRY_METHODS = %w[get head options].freeze
+      RETRYABLE_ERRORS = ["http timeout", "http error"].freeze
+      RETRYABLE_STATUSES = [408, 429].freeze
+      SERVER_ERROR_STATUSES = (500..599)
+      SUCCESS_STATUSES = (200..299)
+      REDIRECT_STATUSES = (300..399)
+      PRESERVE_METHOD_REDIRECT_STATUSES = [307, 308].freeze
       MAX_HEADER_BYTES = 32.kilobytes
       HEADER_NAME = /\A[A-Za-z0-9!#$%&'*+\-.^_`|~]+\z/
       RESTRICTED_HEADERS = %w[
@@ -285,16 +299,16 @@ class Code
         options = arguments.third.to_code
         options = Dictionary.new if options.nothing?
         delays = options.code_get("retry_delays")
-        delays = delays.nothing? ? [] : delays.as_json
-        unless delays.size <= 5 &&
+        delays = delays.nothing? ? DEFAULT_RETRY_DELAYS : delays.as_json
+        unless delays.size <= MAX_RETRIES &&
                  delays.all? do |delay|
                    delay.is_a?(::Numeric) && delay.finite? &&
-                     delay.between?(0, 60)
+                     delay.between?(0, MAX_RETRY_DELAY)
                  end
           raise ::Code::Error, "http: invalid retry delays"
         end
 
-        safe = %w[get head options].include?(arguments.first.to_s.downcase)
+        safe = SAFE_RETRY_METHODS.include?(arguments.first.to_s.downcase)
         safe ||= options.code_get("retry_unsafe").truthy?
         safe ||=
           (options.code_get("headers").as_json || {}).any? do |key, value|
@@ -306,7 +320,7 @@ class Code
           begin
             response = code_request(*arguments, redirects: redirects)
           rescue ::Code::Error => e
-            unless ["http timeout", "http error"].include?(e.message) &&
+            unless RETRYABLE_ERRORS.include?(e.message) &&
                      attempt < delays.size
               raise
             end
@@ -316,7 +330,7 @@ class Code
             next
           end
           status = response.code_get("code").raw
-          unless [408, 429].include?(status) || status.between?(500, 599)
+          unless RETRYABLE_STATUSES.include?(status) || SERVER_ERROR_STATUSES.cover?(status)
             return response
           end
           return response if attempt >= delays.size
@@ -333,7 +347,7 @@ class Code
               delay = delays[attempt]
             end
           end
-          return response if delay > 60
+          return response if delay > MAX_RETRY_DELAY
 
           sleep(delay)
           attempt += 1
@@ -349,7 +363,7 @@ class Code
           redirects =
             (
               if options.code_get("redirects").nothing?
-                0
+                DEFAULT_REDIRECTS
               else
                 options.code_get("redirects").raw
               end
@@ -358,12 +372,13 @@ class Code
         max_retries =
           (
             if options.code_get("max_retries").nothing?
-              0
+              DEFAULT_MAX_RETRIES
             else
               options.code_get("max_retries").raw
             end
           )
-        unless redirects.between?(0, 10) && max_retries.between?(0, 5)
+        unless redirects.between?(0, MAX_REDIRECTS) &&
+                 max_retries.between?(0, MAX_RETRIES)
           raise ::Code::Error, "http: invalid request limits"
         end
 
@@ -399,7 +414,7 @@ class Code
 
         max_bytes = options.code_get("max_bytes")
         max_bytes = max_bytes.nothing? ? nil : max_bytes.raw
-        if max_bytes && !max_bytes.between?(1, 50.megabytes)
+        if max_bytes && !max_bytes.between?(1, MAX_RESPONSE_BYTES)
           raise ::Code::Error, "http: invalid response size limit"
         end
 
@@ -472,12 +487,12 @@ class Code
         code = response.code.to_i
         location = response["location"].to_s
 
-        if (300..399).cover?(code) && location.present? && redirects.positive?
+        if REDIRECT_STATUSES.cover?(code) && location.present? && redirects.positive?
           new_uri = parse_uri(::URI.join(uri, location).to_s)
           ::Code::Network.validate_public_uri!(new_uri, service: "http")
 
           redirect_options = options.as_json.except("query")
-          redirect_method = [307, 308].include?(code) ? verb : "get"
+          redirect_method = PRESERVE_METHOD_REDIRECT_STATUSES.include?(code) ? verb : "get"
           unless same_origin?(new_uri, uri)
             unless %w[get head].include?(redirect_method)
               raise ::Code::Error,
@@ -509,8 +524,8 @@ class Code
             :headers => response_headers,
             :method => verb,
             :url => url,
-            "success?" => code.between?(200, 299),
-            "redirect?" => code.between?(300, 399),
+            "success?" => SUCCESS_STATUSES.cover?(code),
+            "redirect?" => REDIRECT_STATUSES.cover?(code),
             :request => {
               method: verb,
               url: url,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: code-ruby
 version: !ruby/object:Gem::Version
-  version: 5.2.4
+  version: 6.0.0
 platform: ruby
 authors:
 - Dorian Marié