code-ruby 4.0.0 → 4.0.2

data/lib/code/object/http.rb

@@ -5,7 +5,8 @@ class Code
     class Http < Object
       CLASS_DOCUMENTATION = {
         name: "Http",
-        description: "sends http requests and returns dictionaries with request and response details.",
+        description:
+          "sends http requests and returns dictionaries with request and response details.",
         examples: [
           "Http.get(\"http://httpbin.org/status/200\").success?",
           "Http.post(\"http://httpbin.org/status/200\", body: :hello).request.body",
@@ -15,7 +16,8 @@ class Code
       CLASS_FUNCTIONS = {
         "get" => {
           name: "get",
-          description: "sends a get request and returns the response dictionary.",
+          description:
+            "sends a get request and returns the response dictionary.",
           examples: [
             "Http.get(\"http://httpbin.org/status/200\").code",
             "Http.get(\"http://httpbin.org/status/200\", query: { q: :ruby }).url",
@@ -24,7 +26,8 @@ class Code
         },
         "head" => {
           name: "head",
-          description: "sends a head request and returns the response dictionary.",
+          description:
+            "sends a head request and returns the response dictionary.",
           examples: [
             "Http.head(\"http://httpbin.org/status/200\").code",
             "Http.head(\"http://httpbin.org/status/204\").status",
@@ -33,7 +36,8 @@ class Code
         },
         "post" => {
           name: "post",
-          description: "sends a post request with optional body, form data, and headers.",
+          description:
+            "sends a post request with optional body, form data, and headers.",
           examples: [
             "Http.post(\"http://httpbin.org/status/200\", body: :hello).request.body",
             "Http.post(\"http://httpbin.org/status/200\", data: { a: 1 }).success?",
@@ -51,7 +55,8 @@ class Code
         },
         "delete" => {
           name: "delete",
-          description: "sends a delete request and returns the response dictionary.",
+          description:
+            "sends a delete request and returns the response dictionary.",
           examples: [
             "Http.delete(\"http://httpbin.org/status/200\").success?",
             "Http.delete(\"http://httpbin.org/status/200\", body: :hello).request.body",
@@ -60,7 +65,8 @@ class Code
         },
         "options" => {
           name: "options",
-          description: "sends an options request and returns the response dictionary.",
+          description:
+            "sends an options request and returns the response dictionary.",
           examples: [
             "Http.options(\"http://httpbin.org/status/200\").method",
             "Http.options(\"http://httpbin.org/status/204\").code",
@@ -69,7 +75,8 @@ class Code
         },
         "trace" => {
           name: "trace",
-          description: "sends a trace request and returns the response dictionary.",
+          description:
+            "sends a trace request and returns the response dictionary.",
           examples: [
             "Http.trace(\"http://httpbin.org/status/200\").method",
             "Http.trace(\"http://httpbin.org/status/204\").code",
@@ -184,8 +191,6 @@ class Code
         network_authentication_required: 511
       }.freeze
       DEFAULT_TIMEOUT = 1.hour.to_f
-      MAX_REQUEST_BYTES = ::Code::MAX_INPUT_BYTES
-      MAX_RESPONSE_BYTES = ::Code::MAX_INPUT_BYTES
       MAX_HEADER_BYTES = 32.kilobytes
       HEADER_NAME = /\A[A-Za-z0-9!#$%&'*+\-.^_`|~]+\z/
       RESTRICTED_HEADERS = %w[
@@ -284,15 +289,9 @@ class Code
         write_timeout = options.code_get("write_timeout")
         query = options.code_get("query").raw || {}
         query = query.to_a.flatten.map(&:to_s).each_slice(2).to_h.to_query
-        validate_payload_size!(username, label: "http username")
-        validate_payload_size!(password, label: "http password")
-        validate_payload_size!(query, label: "http query")
-        validate_payload_size!(body, label: "http request body")
-        validate_payload_size!(data.as_json.to_query, label: "http form data") if data.present?
 
         url = original_url
         url = "#{url}?#{query}" if query.present?
-        validate_payload_size!(url, label: "http url")
 
         if username.present? || password.present?
           authorization = ::Base64.strict_encode64("#{username}:#{password}")
@@ -307,12 +306,9 @@ class Code
         http.ipaddr = resolved_ip
         http.use_ssl = true if uri.scheme == "https"
         default_timeout = http_timeout(timeout, DEFAULT_TIMEOUT)
-        open_timeout_value =
-          http_timeout(open_timeout, default_timeout)
-        read_timeout_value =
-          http_timeout(read_timeout, default_timeout)
-        write_timeout_value =
-          http_timeout(write_timeout, default_timeout)
+        open_timeout_value = http_timeout(open_timeout, default_timeout)
+        read_timeout_value = http_timeout(read_timeout, default_timeout)
+        write_timeout_value = http_timeout(write_timeout, default_timeout)
 
         http.open_timeout = open_timeout_value if open_timeout_value
         http.read_timeout = read_timeout_value if read_timeout_value
@@ -350,10 +346,6 @@ class Code
               validate_response_headers!(http_response)
 
               http_response.read_body do |chunk|
-                if response_body.bytesize + chunk.bytesize > MAX_RESPONSE_BYTES
-                  raise ::Code::Error, "http response is too large"
-                end
-
                 response_body << chunk
               end
             end
@@ -383,27 +375,26 @@ class Code
         else
           status = STATUS_CODES.key(code) || :ok
           response_headers = response.each_header.to_h
-          request_headers = request.to_hash.transform_values do |values|
-            List.new(values)
-          end
+          request_headers =
+            request.to_hash.transform_values { |values| List.new(values) }
           body = response_body.to_s
 
           Dictionary.new(
-            code: code,
-            status: status,
-            body: body,
-            headers: response_headers,
-            method: verb,
-            url: url,
+            :code => code,
+            :status => status,
+            :body => body,
+            :headers => response_headers,
+            :method => verb,
+            :url => url,
             "success?" => code.between?(200, 299),
             "redirect?" => code.between?(300, 399),
-            request: {
+            :request => {
               method: verb,
               url: url,
               headers: request_headers,
               body: request.body.to_s
             },
-            response: {
+            :response => {
               code: code,
               status: status,
               headers: response_headers,
@@ -421,7 +412,8 @@ class Code
 
       def self.same_origin?(new_uri, original_uri)
         new_uri.scheme == original_uri.scheme &&
-          new_uri.hostname.to_s.downcase == original_uri.hostname.to_s.downcase &&
+          new_uri.hostname.to_s.downcase ==
+            original_uri.hostname.to_s.downcase &&
           new_uri.port == original_uri.port
       end
 
@@ -472,12 +464,6 @@ class Code
         raise ::Code::Error, "http response headers are too large"
       end
 
-      def self.validate_payload_size!(value, label:)
-        return if value.to_s.bytesize <= MAX_REQUEST_BYTES
-
-        raise ::Code::Error, "#{label} is too large"
-      end
-
       def self.http_timeout(value, default)
         ::Code.normalize_timeout!(value.nothing? ? default : value)
       end

data/lib/code/object/ics.rb

@@ -5,7 +5,8 @@ class Code
     class Ics < Object
       CLASS_DOCUMENTATION = {
         name: "Ics",
-        description: "parses icalendar text and exposes events as dictionaries.",
+        description:
+          "parses icalendar text and exposes events as dictionaries.",
         examples: [
           "Ics.parse(\"BEGIN:VCALENDAR\\nBEGIN:VEVENT\\nSUMMARY:meet\\nEND:VEVENT\\nEND:VCALENDAR\")",
           "Ics.parse(\"BEGIN:VCALENDAR\\nVERSION:2.0\\nEND:VCALENDAR\")",
@@ -43,8 +44,6 @@ class Code
         geo
       ].freeze
       MISSING_ATTRIBUTE = Object.new.freeze
-      MAX_EVENTS = 10_000
-      MAX_NESTING = 50
 
       def self.call(**args)
         code_operator = args.fetch(:operator, nil).to_code
@@ -62,10 +61,8 @@ class Code
 
       def self.code_parse(value)
         source = value.to_code.raw
-        ::Code.ensure_input_size!(source, label: "ics")
         calendars = ::Icalendar::Calendar.parse(source)
         events = calendars.flat_map(&:events)
-        raise Error, "ics has too many events" if events.size > MAX_EVENTS
 
         events.map { |event| serialize_event(event) }.to_code
       rescue ::Code::Error
@@ -127,9 +124,7 @@ class Code
         MISSING_ATTRIBUTE
       end
 
-      def self.serialize_value(value, depth: 0)
-        raise Error, "ics is too deeply nested" if depth > MAX_NESTING
-
+      def self.serialize_value(value)
         case value
         when nil
           nil
@@ -138,17 +133,15 @@ class Code
         when ::Symbol, ::Integer, ::Float, ::BigDecimal, true, false
           value
         when ::Array
-          value.map { |item| serialize_value(item, depth: depth + 1) }
+          value.map { |item| serialize_value(item) }
         when ::Hash
-          value.transform_values do |item|
-            serialize_value(item, depth: depth + 1)
-          end
+          value.transform_values { |item| serialize_value(item) }
         else
           serialized_date = serialize_date_like(value)
           return serialized_date unless serialized_date.nil?
 
           if value.is_a?(::Icalendar::Value)
-            serialize_value(value.value, depth: depth + 1)
+            serialize_value(value.value)
           else
             normalize_string(value.to_s)
           end

data/lib/code/object/identifier_list.rb

@@ -5,7 +5,8 @@ class Code
     class IdentifierList < List
       CLASS_DOCUMENTATION = {
         name: "IdentifierList",
-        description: "stores ordered identifier names used to assign nested values.",
+        description:
+          "stores ordered identifier names used to assign nested values.",
         examples: [
           "IdentifierList",
           "IdentifierList.new([:user, :name]).first",
@@ -92,11 +93,13 @@ class Code
           end
 
         if assignment_operator == "="
-          receiver.call(
-            **args,
-            operator: "#{raw.last}=",
-            arguments: Object::List.new([code_value])
-          ).tap { persist_class_receiver(receiver, args.fetch(:context)) }
+          receiver
+            .call(
+              **args,
+              operator: "#{raw.last}=",
+              arguments: Object::List.new([code_value])
+            )
+            .tap { persist_class_receiver(receiver, args.fetch(:context)) }
         else
           next_value =
             receiver.call(
@@ -109,11 +112,13 @@ class Code
               arguments: Object::List.new([code_value])
             )
 
-          receiver.call(
-            **args,
-            operator: "#{raw.last}=",
-            arguments: Object::List.new([next_value])
-          ).tap { persist_class_receiver(receiver, args.fetch(:context)) }
+          receiver
+            .call(
+              **args,
+              operator: "#{raw.last}=",
+              arguments: Object::List.new([next_value])
+            )
+            .tap { persist_class_receiver(receiver, args.fetch(:context)) }
         end
       end