biryani 0.0.9 → 0.0.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 33f21fe89774fbe47cfaa0f36bb1eaaabc333701ffce568c410818aa86c6441d
-  data.tar.gz: 290247cccbfbf21eb1d6d4e93b1f39a4cfb5dbe26fcef278f9804f95221e7ec1
+  metadata.gz: 1aa3fb4dd49abcba22df142cd67d38c5eedc6c8beb6ca8fbcb130147c7e7506a
+  data.tar.gz: 54bd5a493c027047eeaada80b4ff938ab806c29b80069a6cc112a54f5bb78efc
 SHA512:
-  metadata.gz: eed8eb6993bdc77026e12b52836a0970bc6ed6ba024f05bb278049e38ddf9528dadea31c8d42660f0d0c38da08295ac7343702436512d0de148f4f6d82205811
-  data.tar.gz: abffb2767adcddc3556a22367a7a67c78cbc636de1dfa794b65f421c5ac15c30a546e3641edf607bce8119c3c2420856700bbc19439883fc0a74b4fcb3623bc6
+  metadata.gz: 3a8c93ee8ce069cfe77854f7792abadf528addc7e258cd55c84f1b8b2da0b25b278b768e3c8b41a0897a9adc1ee497a9cac4d2ced4c3dd5c644d17f9eb1327c8
+  data.tar.gz: 2909206008b7794ce6ec3a7c850acf66c475c33bc543d55defd9222d8f606fb661d31b99443cc4ba7a0d3d1da920d3a85209d0f8e0cca3de1cd611ccc43634ac

data/.ruby-version

@@ -1 +1 @@
-4.0.1
+4.0.2

data/README.md

@@ -32,8 +32,8 @@ port = ARGV[0] || 8888
 socket = TCPServer.new(port)
 
 server = Biryani::Server.new(
-  # @param _req [Biryani::HTTPRequest]
-  # @param res [Biryani::HTTPResponse]
+  # @param _req [Biryani::HTTP::Request]
+  # @param res [Biryani::HTTP::Response]
   Ractor.shareable_proc do |_req, res|
     res.status = 200
     res.content = 'Hello, world!'

data/example/echo.rb

@@ -9,8 +9,8 @@ port = ARGV[0] || 8888
 socket = TCPServer.new(port)
 
 server = Biryani::Server.new(
-  # @param req [Biryani::HTTPRequest]
-  # @param res [Biryani::HTTPResponse]
+  # @param req [Biryani::HTTP::Request]
+  # @param res [Biryani::HTTP::Response]
   Ractor.shareable_proc do |req, res|
     res.status = 200
     res.content = if req.method.upcase == 'POST'

data/example/hello_world.rb

@@ -9,8 +9,8 @@ port = ARGV[0] || 8888
 socket = TCPServer.new(port)
 
 server = Biryani::Server.new(
-  # @param _req [Biryani::HTTPRequest]
-  # @param res [Biryani::HTTPResponse]
+  # @param _req [Biryani::HTTP::Request]
+  # @param res [Biryani::HTTP::Response]
   Ractor.shareable_proc do |_req, res|
     res.status = 200
     res.content = 'Hello, world!'

data/lib/biryani/connection.rb

@@ -19,7 +19,7 @@ module Biryani
 
     # proc [Proc]
     def initialize(proc)
-      @sock = nil # Ractor
+      @sock = nil # Ractor::Port
       @proc = proc
       @streams_ctx = StreamsContext.new(proc)
       @encoder = HPACK::Encoder.new(4_096)
@@ -43,10 +43,10 @@ module Biryani
       self.class.do_send(io, Frame::Settings.new(false, 0, {}), true)
 
       recv_loop(io.clone)
-      send_loop(io)
+      select_loop(io)
     rescue StandardError => e
       puts e.backtrace
-      self.class.do_send(io, Frame::Goaway.new(0, @streams_ctx.last_stream_id, ErrorCode::INTERNAL_ERROR, 'internal error'), true)
+      self.class.do_send(io, Frame::Goaway.new(@streams_ctx.last_stream_id, ErrorCode::INTERNAL_ERROR, 'internal error'), true)
     ensure
       io.close_write
     end
@@ -64,58 +64,59 @@ module Biryani
     end
 
     # @param io [IO]
-    # rubocop: disable Metrics/AbcSize
-    # rubocop: disable Metrics/BlockLength
-    # rubocop: disable Metrics/CyclomaticComplexity
-    # rubocop: disable Metrics/MethodLength
-    # rubocop: disable Metrics/PerceivedComplexity
-    def send_loop(io)
+    def select_loop(io)
       loop do
         break if @sock.closed? && @streams_ctx.empty?
 
         port, obj = Ractor.select(@sock, @streams_ctx.tx)
         if port == @sock
-          if Biryani.err?(obj)
-            reply_frame = Biryani.unwrap(obj, @streams_ctx.last_stream_id)
-            self.class.do_send(io, reply_frame, true)
-            close if self.class.transition_stream_state_send(reply_frame, @streams_ctx)
-          elsif obj.length > @settings[SettingsID::SETTINGS_MAX_FRAME_SIZE]
-            self.class.do_send(io, Frame::Goaway.new(0, @streams_ctx.last_stream_id, ErrorCode::FRAME_SIZE_ERROR, 'payload length greater than SETTINGS_MAX_FRAME_SIZE'), true)
-            close
-          else
-            recv_dispatch(obj).each do |frame|
-              reply_frame = Biryani.unwrap(frame, @streams_ctx.last_stream_id)
-              self.class.do_send(io, reply_frame, true)
-              if reply_frame.f_type == FrameType::WINDOW_UPDATE && reply_frame.stream_id.zero?
-                @recv_window.increase!(reply_frame.window_size_increment)
-              elsif reply_frame.f_type == FrameType::WINDOW_UPDATE
-                @streams_ctx[reply_frame.stream_id].recv_window.increase!(reply_frame.window_size_increment)
-              end
-
-              close if self.class.transition_stream_state_send(reply_frame, @streams_ctx)
-            end
-          end
+          recv_dispatch(io, obj)
         else
           res, stream_id = obj
-          fragment, data = self.class.http_response(res, @encoder)
-          max_frame_size = @peer_settings[SettingsID::SETTINGS_MAX_FRAME_SIZE]
-          self.class.send_headers(io, stream_id, fragment, data.empty?, max_frame_size, @streams_ctx)
-          self.class.send_data(io, stream_id, data, @send_window, max_frame_size, @streams_ctx, @data_buffer) unless data.empty?
+          handle_response(io, res, stream_id)
         end
 
         break if closed?
       end
     end
+
+    # @param io [IO]
+    # @param obj [Object]
+    #
+    # @return [Array<Object>, Array<ConnectionError>, Array<StreamError>] frames or errors
+    # rubocop: disable Metrics/AbcSize
+    # rubocop: disable Metrics/CyclomaticComplexity
+    # rubocop: disable Metrics/PerceivedComplexity
+    def recv_dispatch(io, obj)
+      if Biryani.err?(obj)
+        reply_frame = Biryani.unwrap(obj, @streams_ctx.last_stream_id)
+        self.class.do_send(io, reply_frame, true)
+        close if self.class.transition_stream_state_send(reply_frame, @streams_ctx)
+      elsif obj.length > @settings[SettingsID::SETTINGS_MAX_FRAME_SIZE]
+        self.class.do_send(io, Frame::Goaway.new(@streams_ctx.last_stream_id, ErrorCode::FRAME_SIZE_ERROR, 'payload length greater than SETTINGS_MAX_FRAME_SIZE'), true)
+        close
+      else
+        do_recv_dispatch(obj).each do |frame|
+          reply_frame = Biryani.unwrap(frame, @streams_ctx.last_stream_id)
+          self.class.do_send(io, reply_frame, true)
+          if reply_frame.f_type == FrameType::WINDOW_UPDATE && reply_frame.stream_id.zero?
+            @recv_window.increase!(reply_frame.window_size_increment)
+          elsif reply_frame.f_type == FrameType::WINDOW_UPDATE
+            @streams_ctx[reply_frame.stream_id].recv_window.increase!(reply_frame.window_size_increment)
+          end
+
+          close if self.class.transition_stream_state_send(reply_frame, @streams_ctx)
+        end
+      end
+    end
     # rubocop: enable Metrics/AbcSize
-    # rubocop: enable Metrics/BlockLength
     # rubocop: enable Metrics/CyclomaticComplexity
-    # rubocop: enable Metrics/MethodLength
     # rubocop: enable Metrics/PerceivedComplexity
 
     # @param frame [Object]
     #
     # @return [Array<Object>, Array<ConnectionError>, Array<StreamError>] frames or errors
-    def recv_dispatch(frame)
+    def do_recv_dispatch(frame)
       receiving_continuation_stream_id = @streams_ctx.receiving_continuation_stream_id
       return [ConnectionError.new(ErrorCode::PROTOCOL_ERROR, "invalid frame type #{format('0x%02x', typ)} for stream identifier #{format('0x%02x', stream_id)}")] \
         if !receiving_continuation_stream_id.nil? && frame.stream_id != receiving_continuation_stream_id
@@ -223,6 +224,16 @@ module Biryani
     # rubocop: enable Metrics/CyclomaticComplexity
     # rubocop: enable Metrics/MethodLength
 
+    # @param io [IO]
+    # @param res [HTTP::Response]
+    # @param stream_id [Integer]
+    def handle_response(io, res, stream_id)
+      fragment, data = self.class.http_response(res, @encoder)
+      max_frame_size = @peer_settings[SettingsID::SETTINGS_MAX_FRAME_SIZE]
+      self.class.send_headers(io, stream_id, fragment, data.empty?, max_frame_size, @streams_ctx)
+      self.class.send_data(io, stream_id, data, @send_window, max_frame_size, @streams_ctx, @data_buffer) unless data.empty?
+    end
+
     def close
       @closed = true
     end
@@ -449,26 +460,26 @@ module Biryani
     # @param content [String]
     # @param decoder [Decoder]
     #
-    # @return [HTTPRequest, ConnectionError]
+    # @return [HTTP::Request, ConnectionError]
     def self.http_request(fragment, content, decoder)
       obj = decoder.decode(fragment)
       return obj if Biryani.err?(obj)
 
       fields = obj
-      builder = HTTPRequestBuilder.new
+      builder = HTTP::RequestBuilder.new
       err = builder.fields(fields)
       return err unless err.nil?
 
       builder.build(content)
     end
 
-    # @param res [HTTPResponse]
+    # @param res [HTTP::Response]
     # @param encoder [Encoder]
     #
     # @return [String] fragment
     # @return [String] data
     def self.http_response(res, encoder)
-      HTTPResponseParser.new(res).parse(encoder)
+      HTTP::ResponseParser.new(res).parse(encoder)
     end
 
     # @return [Hash<Integer, Integer>]

data/lib/biryani/connection_error.rb

@@ -11,7 +11,7 @@ module Biryani
     #
     # @return [Goaway]
     def goaway(last_stream_id)
-      Frame::Goaway.new(0, last_stream_id, @code, @debug)
+      Frame::Goaway.new(last_stream_id, @code, @debug)
     end
   end
 end

data/lib/biryani/frame/goaway.rb

@@ -6,9 +6,9 @@ module Biryani
       # @param last_stream_id [Integer]
       # @param error_code [Integer]
       # @param debug [String]
-      def initialize(stream_id, last_stream_id, error_code, debug)
+      def initialize(last_stream_id, error_code, debug)
         @f_type = FrameType::GOAWAY
-        @stream_id = stream_id
+        @stream_id = 0
         @last_stream_id = last_stream_id
         @error_code = error_code
         @debug = debug
@@ -33,11 +33,13 @@ module Biryani
       #
       # @return [Goaway]
       def self.read(s, _flags, stream_id)
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'invalid frame') unless stream_id.zero?
+
         io = IO::Buffer.for(s)
         last_stream_id, error_code = io.get_values(%i[U32 U32], 0)
         debug = io.get_string(8)
 
-        Goaway.new(stream_id, last_stream_id, error_code, debug)
+        Goaway.new(last_stream_id, error_code, debug)
       end
     end
   end

data/lib/biryani/http/error.rb

@@ -0,0 +1,10 @@
+module Biryani
+  module HTTP
+    module Error
+      # Generic error, common for all classes under Biryani::HTTP::Error module.
+      class Error < StandardError; end
+
+      class InvalidHTTPResponseError < Error; end
+    end
+  end
+end

data/lib/biryani/http/request.rb

@@ -0,0 +1,97 @@
+module Biryani
+  module HTTP
+    class Request
+      attr_accessor :method, :uri, :fields, :content
+
+      # @param method [String]
+      # @param uri [URI]
+      # @param fields [Hash<String, Array<String>>]
+      # @param content [String]
+      def initialize(method, uri, fields, content)
+        @method = method
+        @uri = uri
+        @fields = fields
+        @content = content
+      end
+
+      # @return [Array<String>, nil]
+      def trailers
+        # https://datatracker.ietf.org/doc/html/rfc9110#section-6.6.2-4
+        keys = (@fields['trailer'] || []).flat_map { |x| x.split(',').map(&:strip) }
+        @fields.slice(*keys)
+      end
+    end
+
+    class RequestBuilder
+      PSEUDO_HEADER_FIELDS = [':authority', ':method', ':path', ':scheme'].freeze
+      Ractor.make_shareable(PSEUDO_HEADER_FIELDS)
+
+      def initialize
+        @h = {}
+      end
+
+      # @param name [String]
+      # @param value [String]
+      #
+      # @return [nil, ConnectioError]
+      # rubocop: disable Metrics/AbcSize
+      # rubocop: disable Metrics/CyclomaticComplexity
+      # rubocop: disable Metrics/PerceivedComplexity
+      def field(name, value)
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'field name has uppercase letter') if name.downcase != name
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'unknown pseudo-header field name') if name[0] == ':' && !PSEUDO_HEADER_FIELDS.include?(name)
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'appear pseudo-header fields after regular fields') if name[0] == ':' && @h.any? { |name_, _| name_[0] != ':' }
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'duplicated pseudo-header fields') if PSEUDO_HEADER_FIELDS.include?(name) && @h.key?(name)
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, "invalid `#{name}` field") if PSEUDO_HEADER_FIELDS.include?(name) && value.empty?
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'connection-specific field is forbidden') if name == 'connection'
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, '`TE` field has a value other than `trailers`') if name == 'te' && value != 'trailers'
+
+        @h[name] = [] unless @h.key?(name)
+        @h[name] << value
+
+        nil
+      end
+      # rubocop: enable Metrics/AbcSize
+      # rubocop: enable Metrics/CyclomaticComplexity
+      # rubocop: enable Metrics/PerceivedComplexity
+
+      # @param arr [Array]
+      #
+      # @return [nil, ConnectioError]
+      def fields(arr)
+        arr.each do |name, value|
+          err = field(name, value)
+          return err unless err.nil?
+        end
+
+        nil
+      end
+
+      # @param s [String]
+      #
+      # @return [Request]
+      def build(s)
+        # `Ractor.send(req, move: true)` moves entries in HPACK::DynamicTable; therefore, a `dup` call is required.
+        h = @h.transform_values { |x| x.map(&:dup) }
+        self.class.build(h, s)
+      end
+
+      # @param h [Hash<String, Array<String>>]
+      # @param s [String]
+      #
+      # @return [Request, ConnectionError]
+      def self.build(h, s)
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'missing pseudo-header fields') unless PSEUDO_HEADER_FIELDS.all? { |x| h.key?(x) }
+        return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'invalid content-length') if h.key?('content-length') && !s.empty? && s.length != h['content-length'].to_i
+
+        scheme = h[':scheme'][0]
+        domain = h[':authority'][0]
+        path = h[':path'][0]
+        uri = URI("#{scheme}://#{domain}#{path}")
+        method = h[':method'][0]
+        h['cookie'] = [h['cookie'].join('; ')] if h.key?('cookie')
+        Request.new(method, uri, h, s)
+      end
+    end
+  end
+end

data/lib/biryani/http/response.rb

@@ -0,0 +1,68 @@
+module Biryani
+  module HTTP
+    class Response
+      FORBIDDEN_KEY_CHARS = (0x00..0x20).chain([0x3a]).chain(0x41..0x5a).chain(0x7f..0xff).to_a.freeze
+      FORBIDDEN_VALUE_CHARS = [0x00, 0x0a, 0x0d].freeze
+
+      attr_accessor :status, :fields, :content
+
+      # @param status [Integer]
+      # @param fields [Hash]
+      # @param content [String, nil]
+      def initialize(status, fields, content)
+        @status = status
+        @fields = fields
+        @content = content
+      end
+
+      # @raise [InvalidHTTPResponseError]
+      # rubocop: disable Metrics/CyclomaticComplexity
+      def validate
+        # https://datatracker.ietf.org/doc/html/rfc9113#section-8.2.1
+        raise Error::InvalidHTTPResponseError, 'invalid HTTP status' if @status < 100 || @status >= 600
+        raise Error::InvalidHTTPResponseError, 'HTTP field name contains invalid characters' if (@fields.keys.join.downcase.bytes.uniq & FORBIDDEN_KEY_CHARS).any?
+        raise Error::InvalidHTTPResponseError, 'HTTP field value contains NUL, LF or CR' if (@fields.values.join.bytes.uniq & FORBIDDEN_VALUE_CHARS).any?
+        raise Error::InvalidHTTPResponseError, 'HTTP field value starts/ends with SP or HTAB' if @fields.values.filter { |s| s.start_with?("\t", ' ') || s.end_with?("\t", ' ') }.any?
+      end
+      # rubocop: enable Metrics/CyclomaticComplexity
+
+      def self.default
+        Response.new(0, {}, nil)
+      end
+
+      def self.internal_server_error
+        Response.new(500, {}, 'Internal Server Error')
+      end
+    end
+
+    class ResponseParser
+      # @param res [Response]
+      def initialize(res)
+        @res = res
+      end
+
+      # @return [Array]
+      def fields
+        fields = [[':status', @res.status.to_s]]
+        @res.fields.each do |name, value|
+          fields << [name.to_s.downcase, value.to_s]
+        end
+
+        fields
+      end
+
+      # @return [String]
+      def content
+        @res.content
+      end
+
+      # @param encoder [Encoder]
+      #
+      # @return [String] fragment
+      # @return [String] data
+      def parse(encoder)
+        [encoder.encode(fields), content]
+      end
+    end
+  end
+end

data/lib/biryani/http.rb

@@ -0,0 +1,3 @@
+require_relative 'http/error'
+require_relative 'http/request'
+require_relative 'http/response'

data/lib/biryani/stream.rb

@@ -2,20 +2,20 @@ module Biryani
   class Stream
     attr_accessor :rx
 
-    # @param tx [Port]
+    # @param tx [Ractor::Port]
     # @param stream_id [Integer]
     # @param proc [Proc]
     def initialize(tx, stream_id, proc)
       @rx = Ractor.new(tx, stream_id, proc) do |tx, stream_id, proc|
         unless (req = Ractor.recv).nil?
-          res = HTTPResponse.default
+          res = HTTP::Response.default
 
           begin
             proc.call(req, res)
             res.validate
           rescue StandardError => e
             puts e.backtrace
-            res = HTTPResponse.internal_server_error
+            res = HTTP::Response.internal_server_error
           end
 
           tx.send([res, stream_id], move: true)

data/lib/biryani/streams_context.rb

@@ -117,7 +117,7 @@ module Biryani
       @state = State.new
     end
 
-    # @param req [HTTPRequest]
+    # @param req [HTTP::Request]
     def <<(req)
       @stream.rx.send(req, move: true)
     end

data/lib/biryani/version.rb

@@ -1,3 +1,3 @@
 module Biryani
-  VERSION = '0.0.9'.freeze
+  VERSION = '0.0.10'.freeze
 end

data/lib/biryani.rb

@@ -3,11 +3,9 @@ require 'uri'
 require_relative 'biryani/connection_error'
 require_relative 'biryani/connection'
 require_relative 'biryani/data_buffer'
-require_relative 'biryani/error'
 require_relative 'biryani/frame'
 require_relative 'biryani/hpack'
-require_relative 'biryani/http_request'
-require_relative 'biryani/http_response'
+require_relative 'biryani/http'
 require_relative 'biryani/server'
 require_relative 'biryani/state'
 require_relative 'biryani/stream_error'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: biryani
 version: !ruby/object:Gem::Version
-  version: 0.0.9
+  version: 0.0.10
 platform: ruby
 authors:
 - thekuwayama
@@ -46,7 +46,6 @@ files:
 - lib/biryani/connection.rb
 - lib/biryani/connection_error.rb
 - lib/biryani/data_buffer.rb
-- lib/biryani/error.rb
 - lib/biryani/frame.rb
 - lib/biryani/frame/continuation.rb
 - lib/biryani/frame/data.rb
@@ -70,8 +69,10 @@ files:
 - lib/biryani/hpack/integer.rb
 - lib/biryani/hpack/option.rb
 - lib/biryani/hpack/string.rb
-- lib/biryani/http_request.rb
-- lib/biryani/http_response.rb
+- lib/biryani/http.rb
+- lib/biryani/http/error.rb
+- lib/biryani/http/request.rb
+- lib/biryani/http/response.rb
 - lib/biryani/server.rb
 - lib/biryani/state.rb
 - lib/biryani/stream.rb
@@ -98,7 +99,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 4.0.8
 specification_version: 4
 summary: An HTTP/2 server implemented using Ruby Ractor
 test_files: []

data/lib/biryani/error.rb

@@ -1,8 +0,0 @@
-module Biryani
-  module Error
-    # Generic error, common for all classes under Biryani::Error module.
-    class Error < StandardError; end
-
-    class InvalidHTTPResponseError < Error; end
-  end
-end

data/lib/biryani/http_request.rb

@@ -1,95 +0,0 @@
-module Biryani
-  class HTTPRequest
-    attr_accessor :method, :uri, :fields, :content
-
-    # @param method [String]
-    # @param uri [URI]
-    # @param fields [Hash<String, Array<String>>]
-    # @param content [String]
-    def initialize(method, uri, fields, content)
-      @method = method
-      @uri = uri
-      @fields = fields
-      @content = content
-    end
-
-    # @return [Array<String>, nil]
-    def trailers
-      # https://datatracker.ietf.org/doc/html/rfc9110#section-6.6.2-4
-      keys = (@fields['trailer'] || []).flat_map { |x| x.split(',').map(&:strip) }
-      @fields.slice(*keys)
-    end
-  end
-
-  class HTTPRequestBuilder
-    PSEUDO_HEADER_FIELDS = [':authority', ':method', ':path', ':scheme'].freeze
-    Ractor.make_shareable(PSEUDO_HEADER_FIELDS)
-
-    def initialize
-      @h = {}
-    end
-
-    # @param name [String]
-    # @param value [String]
-    #
-    # @return [nil, ConnectioError]
-    # rubocop: disable Metrics/AbcSize
-    # rubocop: disable Metrics/CyclomaticComplexity
-    # rubocop: disable Metrics/PerceivedComplexity
-    def field(name, value)
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'field name has uppercase letter') if name.downcase != name
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'unknown pseudo-header field name') if name[0] == ':' && !PSEUDO_HEADER_FIELDS.include?(name)
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'appear pseudo-header fields after regular fields') if name[0] == ':' && @h.any? { |name_, _| name_[0] != ':' }
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'duplicated pseudo-header fields') if PSEUDO_HEADER_FIELDS.include?(name) && @h.key?(name)
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, "invalid `#{name}` field") if PSEUDO_HEADER_FIELDS.include?(name) && value.empty?
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'connection-specific field is forbidden') if name == 'connection'
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, '`TE` field has a value other than `trailers`') if name == 'te' && value != 'trailers'
-
-      @h[name] = [] unless @h.key?(name)
-      @h[name] << value
-
-      nil
-    end
-    # rubocop: enable Metrics/AbcSize
-    # rubocop: enable Metrics/CyclomaticComplexity
-    # rubocop: enable Metrics/PerceivedComplexity
-
-    # @param arr [Array]
-    #
-    # @return [nil, ConnectioError]
-    def fields(arr)
-      arr.each do |name, value|
-        err = field(name, value)
-        return err unless err.nil?
-      end
-
-      nil
-    end
-
-    # @param s [String]
-    #
-    # @return [HTTPRequest]
-    def build(s)
-      # `Ractor.send(req, move: true)` moves entries in HPACK::DynamicTable; therefore, a `dup` call is required.
-      h = @h.transform_values { |x| x.map(&:dup) }
-      self.class.http_request(h, s)
-    end
-
-    # @param h [Hash<String, Array<String>>]
-    # @param s [String]
-    #
-    # @return [HTTPRequest, ConnectionError]
-    def self.http_request(h, s)
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'missing pseudo-header fields') unless PSEUDO_HEADER_FIELDS.all? { |x| h.key?(x) }
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'invalid content-length') if h.key?('content-length') && !s.empty? && s.length != h['content-length'].to_i
-
-      scheme = h[':scheme'][0]
-      domain = h[':authority'][0]
-      path = h[':path'][0]
-      uri = URI("#{scheme}://#{domain}#{path}")
-      method = h[':method'][0]
-      h['cookie'] = [h['cookie'].join('; ')] if h.key?('cookie')
-      HTTPRequest.new(method, uri, h, s)
-    end
-  end
-end

data/lib/biryani/http_response.rb

@@ -1,66 +0,0 @@
-module Biryani
-  class HTTPResponse
-    FORBIDDEN_KEY_CHARS = (0x00..0x20).chain([0x3a]).chain(0x41..0x5a).chain(0x7f..0xff).to_a.freeze
-    FORBIDDEN_VALUE_CHARS = [0x00, 0x0a, 0x0d].freeze
-
-    attr_accessor :status, :fields, :content
-
-    # @param status [Integer]
-    # @param fields [Hash]
-    # @param content [String, nil]
-    def initialize(status, fields, content)
-      @status = status
-      @fields = fields
-      @content = content
-    end
-
-    # @raise [InvalidHTTPResponseError]
-    # rubocop: disable Metrics/CyclomaticComplexity
-    def validate
-      # https://datatracker.ietf.org/doc/html/rfc9113#section-8.2.1
-      raise Error::InvalidHTTPResponseError, 'invalid HTTP status' if @status < 100 || @status >= 600
-      raise Error::InvalidHTTPResponseError, 'HTTP field name contains invalid characters' if (@fields.keys.join.downcase.bytes.uniq & FORBIDDEN_KEY_CHARS).any?
-      raise Error::InvalidHTTPResponseError, 'HTTP field value contains NUL, LF or CR' if (@fields.values.join.bytes.uniq & FORBIDDEN_VALUE_CHARS).any?
-      raise Error::InvalidHTTPResponseError, 'HTTP field value starts/ends with SP or HTAB' if @fields.values.filter { |s| s.start_with?("\t", ' ') || s.end_with?("\t", ' ') }.any?
-    end
-    # rubocop: enable Metrics/CyclomaticComplexity
-
-    def self.default
-      HTTPResponse.new(0, {}, nil)
-    end
-
-    def self.internal_server_error
-      HTTPResponse.new(500, {}, 'Internal Server Error')
-    end
-  end
-
-  class HTTPResponseParser
-    # @param res [HTTPResponse]
-    def initialize(res)
-      @res = res
-    end
-
-    # @return [Array]
-    def fields
-      fields = [[':status', @res.status.to_s]]
-      @res.fields.each do |name, value|
-        fields << [name.to_s.downcase, value.to_s]
-      end
-
-      fields
-    end
-
-    # @return [String]
-    def content
-      @res.content
-    end
-
-    # @param encoder [Encoder]
-    #
-    # @return [String] fragment
-    # @return [String] data
-    def parse(encoder)
-      [encoder.encode(fields), content]
-    end
-  end
-end