biryani 0.0.6 → 0.0.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2a6614c6a74a069a08281c0f3846da8512f379e347ce77e3812c7cd22f2506bc
-  data.tar.gz: 5a0f61a32f576c0a693abeb2773490ddcbbdcc573b82dac16a6d3a3e82bf2e0e
+  metadata.gz: 24efc38cdd56ee46ac5e48dcbf52f7ddb73a38c73872c8b05be3d8b26aea5105
+  data.tar.gz: f0d0e70ef0dfc20587711637856f8a7ffe7a310d437d3e53401e81f8ef014391
 SHA512:
-  metadata.gz: 0c0add40f15c4e1fa9110a295f9fda392aae21a6ad4dd062a0d3a31fd5dd208f8a384fbcf9009a6fc29e6a940f4fc755f9da0f3e244f68292ee41323cd10e82f
-  data.tar.gz: 1af73e5c917c095dfcb635fd7c09548b4330114125e00df539356fb270d8f083e76c0d25262652f6781e4b8415fc3c2da91083b0af19b20ac7175a70b5c28bc1
+  metadata.gz: 2c3001f6c1f679690c63bcfe45baa1d85fedf764850c6fa2ca44debe71e89c286ca066715592632b8d19b03a1957f98f2a362aace2e8e171533e3595580a3b8a
+  data.tar.gz: 84a1b0ee18b7eaad1a7f8336ffe912038bd18c8a229c119dac83b7cb9539daee79ab25ea13286db2706e707955b23f6a7801b96c2acf09dbbdc889fc38ddc4a3

data/.github/workflows/conformance.yml

@@ -0,0 +1,46 @@
+name: h2spec
+
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    branches:
+      - '*'
+
+jobs:
+  conformance:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      checks: write
+    strategy:
+      matrix:
+        ruby-version: ['4.0']
+    steps:
+      - uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@ae195bbe749a7cef685ac729197124a48305c1cb # v1.276.0
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+      - name: Install dependencies
+        run: |
+          gem --version
+          gem install bundler
+          bundle --version
+          bundle install
+      - name: Install h2spec
+        run: |
+          set -eu
+          curl -Ls https://github.com/summerwind/h2spec/releases/download/v2.6.0/h2spec_linux_amd64.tar.gz | tar xz
+          mkdir -p $GITHUB_WORKSPACE/bin
+          mv h2spec $GITHUB_WORKSPACE/bin/h2spec
+          chmod +x $GITHUB_WORKSPACE/bin/h2spec
+          echo $GITHUB_WORKSPACE/bin >> $GITHUB_PATH
+      - name: Run conformance test
+        run: bundle exec rake conformance
+      - uses: mikepenz/action-junit-report@a294a61c909bd8a4b563024a2faa28897fd53ebc # v6.1.0
+        if: success()
+        with:
+          report_paths: conformance/reports/h2spec.xml
+          detailed_summary: true

data/.gitignore

@@ -12,6 +12,7 @@
 /rdoc/
 /coverage/
 /spec/reports/
+/conformance/reports/
 /tmp/
 .DS_Store
 Gemfile.lock

data/.rubocop.yml

@@ -10,6 +10,7 @@ Metrics/AbcSize:
 Metrics/BlockLength:
   Exclude:
     - 'spec/**/*'
+    - 'conformance/**/*'
 
 Metrics/ClassLength:
   Max: 200

data/conformance/server_spec.rb

@@ -6,7 +6,7 @@ RSpec.describe Server do
 
     Ractor.new(@tcpserver) do |socket|
       server = Server.new(
-        Ractor.shareable_proc do |_req, res|
+        Ractor.shareable_proc do |_, res|
           res.status = 200
           res.content = 'OK'
         end
@@ -16,9 +16,16 @@ RSpec.describe Server do
     end
   end
 
+  let(:junit_report_file_path) do
+    Dir.mkdir(JUNIT_REPORT_DIR) unless Dir.exist?(JUNIT_REPORT_DIR)
+
+    "#{JUNIT_REPORT_DIR}/h2spec.xml"
+  end
+
   let(:client) do
     which('h2spec')
-    "h2spec --port #{PORT} --verbose"
+
+    "h2spec --port #{PORT} --verbose --junit-report #{junit_report_file_path}"
   end
 
   after do

data/conformance/spec_helper.rb

@@ -7,6 +7,7 @@ include Biryani
 # rubocop: enable Style/MixinUsage
 
 PORT = 8888
+JUNIT_REPORT_DIR = "#{__dir__}/reports".freeze
 
 def which(cmd)
   o, = Open3.capture3("which #{cmd}")

data/lib/biryani/connection.rb

@@ -118,6 +118,10 @@ module Biryani
     #
     # @return [Array<Object>, Array<ConnectionError>, Array<StreamError>] frames or errors
     def 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
+
       if frame.stream_id.zero?
         handle_connection_frame(frame)
       else
@@ -129,13 +133,8 @@ module Biryani
     #
     # @return [Array<Object>, Array<ConnectionError>, Array<StreamError>] frames or errors
     # rubocop: disable Metrics/CyclomaticComplexity
-    # rubocop: disable Metrics/PerceivedComplexity
     def handle_connection_frame(frame)
-      typ = frame.f_type
-      return [ConnectionError.new(ErrorCode::PROTOCOL_ERROR, "invalid frame type #{format('0x%02x', typ)} for stream identifier #{format('0x%02x', stream_id)}")] \
-        if @streams_ctx.receiving_continuation? && ([FrameType::SETTINGS, FrameType::PING, FrameType::WINDOW_UPDATE].include?(typ) || FrameType.unknown?(typ))
-
-      case typ
+      case frame.f_type
       when FrameType::DATA, FrameType::HEADERS, FrameType::PRIORITY, FrameType::RST_STREAM, FrameType::PUSH_PROMISE, FrameType::CONTINUATION
         [ConnectionError.new(ErrorCode::PROTOCOL_ERROR, "invalid frame type #{format('0x%02x', typ)} for stream identifier 0x00")]
       when FrameType::SETTINGS
@@ -167,7 +166,6 @@ module Biryani
       end
     end
     # rubocop: enable Metrics/CyclomaticComplexity
-    # rubocop: enable Metrics/PerceivedComplexity
 
     # @param frame [Object]
     #
@@ -253,7 +251,7 @@ module Biryani
       return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'new stream identifier is less than the existing stream identifiers') if ctx.nil? && streams_ctx.last_stream_id > stream_id
 
       ctx = streams_ctx.new_context(stream_id, send_initial_window_size, recv_initial_window_size) if ctx.nil?
-      obj = ctx.state.transition!(recv_frame, :recv)
+      obj = ctx.state_transition!(recv_frame, :recv)
       return obj if Biryani.err?(obj)
 
       ctx
@@ -275,7 +273,7 @@ module Biryani
         streams_ctx.close_all
         true
       else
-        streams_ctx[stream_id].state.transition!(send_frame, :send) unless stream_id.zero?
+        streams_ctx[stream_id].state_transition!(send_frame, :send) unless stream_id.zero?
         false
       end
     end
@@ -357,7 +355,7 @@ module Biryani
         obj = http_request(ctx.fragment, ctx.content, decoder)
         return obj if Biryani.err?(obj)
 
-        ctx.stream.rx.send(obj, move: true)
+        ctx << obj
       end
 
       window_updates = []
@@ -378,7 +376,7 @@ module Biryani
         obj = http_request(ctx.fragment, ctx.content, decoder)
         return obj if Biryani.err?(obj)
 
-        ctx.stream.rx.send(obj, move: true)
+        ctx << obj
       end
 
       nil
@@ -387,7 +385,7 @@ module Biryani
     # @param _rst_stream [RstStream]
     # @param ctx [StreamContext]
     def self.handle_rst_stream(_rst_stream, ctx)
-      ctx.state.close
+      ctx.close
     end
 
     # @param settings [Settings]

data/lib/biryani/http_request.rb

@@ -4,7 +4,7 @@ module Biryani
 
     # @param method [String]
     # @param uri [URI]
-    # @param fields [Hash<String, String>]
+    # @param fields [Hash<String, Array<String>>]
     # @param content [String]
     def initialize(method, uri, fields, content)
       @method = method
@@ -12,6 +12,13 @@ module Biryani
       @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
@@ -26,6 +33,7 @@ module Biryani
     # @param value [String]
     #
     # @return [nil, ConnectioError]
+    # rubocop: disable Metrics/AbcSize
     # rubocop: disable Metrics/CyclomaticComplexity
     # rubocop: disable Metrics/PerceivedComplexity
     def field(name, value)
@@ -35,17 +43,14 @@ module Biryani
       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'
 
-      # TODO: trailers
-
-      if name == 'cookie' && @h.key?('cookie')
-        @h[name] << "; #{value}"
-      else
-        @h[name] = value
-      end
+      @h[name] = [] unless @h.key?(name)
+      @h[name] << value
 
       nil
     end
+    # rubocop: enable Metrics/AbcSize
     # rubocop: enable Metrics/CyclomaticComplexity
     # rubocop: enable Metrics/PerceivedComplexity
 
@@ -69,16 +74,21 @@ module Biryani
       self.class.http_request(h, s)
     end
 
-    # @param fields [Hash<String, String>]
+    # @param h [Hash<String, Array<String>>]
     # @param s [String]
     #
     # @return [HTTPRequest, ConnectionError]
-    def self.http_request(fields, s)
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'missing pseudo-header fields') unless PSEUDO_HEADER_FIELDS.all? { |x| fields.key?(x) }
-      return ConnectionError.new(ErrorCode::PROTOCOL_ERROR, 'invalid content-length') if fields.key?('content-length') && !s.empty? && s.length != fields['content-length'].to_i
+    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
 
-      uri = URI("#{fields[':scheme']}://#{fields[':authority']}#{fields[':path']}")
-      HTTPRequest.new(fields[':method'], uri, fields.reject { |name, _| PSEUDO_HEADER_FIELDS.include?(name) }, s)
+      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/state.rb

@@ -56,58 +56,82 @@ module Biryani
     # rubocop: disable Metrics/PerceivedComplexity
     def self.next(state, frame, direction)
       typ = frame.f_type
-      case [state, typ, direction]
+      case [state, direction, typ]
       # idle
-      in [:idle, FrameType::HEADERS, :recv] if frame.end_stream? && frame.end_headers?
+      in [:idle, :send, FrameType::PUSH_PROMISE]
+        :reserved_local
+      in [:idle, :recv, FrameType::HEADERS] if frame.end_stream? && frame.end_headers?
         :half_closed_remote
-      in [:idle, FrameType::HEADERS, :recv] if frame.end_headers?
+      in [:idle, :recv, FrameType::HEADERS] if frame.end_headers?
         :receiving_data
-      in [:idle, FrameType::HEADERS, :recv] if frame.end_stream?
+      in [:idle, :recv, FrameType::HEADERS] if frame.end_stream?
         :receiving_continuation
-      in [:idle, FrameType::HEADERS, :recv]
+      in [:idle, :recv, FrameType::HEADERS]
         :receiving_continuation_and_data
-      in [:idle, FrameType::PRIORITY, :recv]
+      in [:idle, :recv, FrameType::PRIORITY]
         state
-      in [:idle, FrameType::PUSH_PROMISE, :send]
-        :reserved_local
-      in [:idle, _, _]
-        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
 
       # receiving_continuation_and_data
-      in [:receiving_continuation_and_data, FrameType::RST_STREAM, _]
-        :closed
-      in [:receiving_continuation_and_data, FrameType::WINDOW_UPDATE, :recv]
+      in [:receiving_continuation_and_data, :recv, FrameType::WINDOW_UPDATE]
         state
-      in [:receiving_continuation_and_data, FrameType::CONTINUATION, :recv] if frame.end_headers?
+      in [:receiving_continuation_and_data, :recv, FrameType::CONTINUATION] if frame.end_headers?
         :receiving_data
-      in [:receiving_continuation_and_data, FrameType::CONTINUATION, :recv]
+      in [:receiving_continuation_and_data, :recv, FrameType::CONTINUATION]
         state
-      in [:receiving_continuation_and_data, _, _]
-        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
+      in [:receiving_continuation_and_data, _, FrameType::RST_STREAM]
+        :closed
 
       # receiving_continuation
-      in [:receiving_continuation, FrameType::RST_STREAM, _]
-        :closed
-      in [:receiving_continuation, FrameType::WINDOW_UPDATE, :recv]
+      in [:receiving_continuation, :recv, FrameType::DATA] if frame.end_stream?
+        :half_closed_remote
+      in [:receiving_continuation, :recv, FrameType::DATA]
+        :receiving_data
+      in [:receiving_continuation, :recv, FrameType::WINDOW_UPDATE]
         state
-      in [:receiving_continuation, FrameType::CONTINUATION, :recv] if frame.end_headers?
+      in [:receiving_continuation, :recv, FrameType::CONTINUATION] if frame.end_headers?
         :half_closed_remote
-      in [:receiving_continuation, FrameType::CONTINUATION, :recv]
+      in [:receiving_continuation, :recv, FrameType::CONTINUATION]
         state
-      in [:receiving_continuation, _, _]
-        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
+      in [:receiving_continuation, _, FrameType::RST_STREAM]
+        :closed
 
       # receiving_data
-      in [:receiving_data, FrameType::DATA, :recv] if frame.end_stream?
+      in [:receiving_data, :recv, FrameType::DATA] if frame.end_stream?
         :half_closed_remote
-      in [:receiving_data, FrameType::DATA, :recv]
+      in [:receiving_data, :recv, FrameType::DATA]
         state
-      in [:receiving_data, FrameType::RST_STREAM, _]
+      in [:receiving_data, :recv, FrameType::HEADERS] if !frame.end_stream?
+        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
+      in [:receiving_data, :recv, FrameType::HEADERS] if frame.end_headers?
+        :half_closed_remote
+      in [:receiving_data, :recv, FrameType::HEADERS]
+        :receiving_trailer_continuation
+      in [:receiving_data, :recv, FrameType::PRIORITY]
+        state
+      in [:receiving_data, _, FrameType::RST_STREAM]
         :closed
-      in [:receiving_data, FrameType::WINDOW_UPDATE, _]
+      in [:receiving_data, _, FrameType::WINDOW_UPDATE]
         state
-      in [:receiving_data, _, _]
+
+      # receiving_trailer_headers
+      in [:receiving_trailer_headers, :recv, FrameType::HEADERS] if !frame.end_stream?
         unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
+      in [:receiving_trailer_headers, :recv, FrameType::HEADERS] if frame.end_headers?
+        :half_closed_remote
+      in [:receiving_trailer_headers, :recv, FrameType::HEADERS]
+        :receiving_trailer_continuation
+      in [:receiving_trailer_headers, :recv, FrameType::WINDOW_UPDATE]
+        state
+
+      # receiving_trailer_continuation
+      in [:receiving_trailer_continuation, :recv, FrameType::WINDOW_UPDATE]
+        state
+      in [:receiving_trailer_continuation, :recv, FrameType::CONTINUATION] if frame.end_headers?
+        :half_closed_remote
+      in [:receiving_trailer_continuation, :recv, FrameType::CONTINUATION]
+        :receiving_trailer_continuation
+      in [:receiving_trailer_continuation, _, FrameType::RST_STREAM]
+        :closed
 
       # reserved_remote
       in [:reserved_remote, _, _]
@@ -120,81 +144,75 @@ module Biryani
         state
 
       # half_closed_remote
-      in [:half_closed_remote, FrameType::HEADERS, :send] if frame.end_stream? && frame.end_headers?
+      in [:half_closed_remote, :send, FrameType::HEADERS] if frame.end_stream? && frame.end_headers?
         :closed
-      in [:half_closed_remote, FrameType::HEADERS, :send] if frame.end_headers?
+      in [:half_closed_remote, :send, FrameType::HEADERS] if frame.end_headers?
         :sending_data
-      in [:half_closed_remote, FrameType::HEADERS, :send] if frame.end_stream?
+      in [:half_closed_remote, :send, FrameType::HEADERS] if frame.end_stream?
         :sending_continuation
-      in [:half_closed_remote, FrameType::HEADERS, :send]
-        :sending_continuation_data
-      in [:half_closed_remote, FrameType::PRIORITY, :recv]
+      in [:half_closed_remote, :send, FrameType::HEADERS]
+        :sending_continuation_and_data
+      in [:half_closed_remote, :recv, FrameType::PRIORITY]
         state
-      in [:half_closed_remote, FrameType::RST_STREAM, _]
+      in [:half_closed_remote, _, FrameType::RST_STREAM]
         :closed
-      in [:half_closed_remote, FrameType::WINDOW_UPDATE, _]
+      in [:half_closed_remote, _, FrameType::WINDOW_UPDATE]
         state
-      in [:half_closed_remote, _, :recv]
-        unexpected(ErrorCode::STREAM_CLOSED, state, typ, direction)
-      in [:half_closed_local, _, :send]
-        unreachable(state, typ, direction)
 
-      # sending_continuation_data
-      in [:sending_continuation_data, FrameType::RST_STREAM, :send]
-        :closed
-      in [:sending_continuation_data, FrameType::WINDOW_UPDATE, :recv]
-        state
-      in [:sending_continuation_data, FrameType::CONTINUATION, :send] if frame.end_headers?
+      # sending_continuation_and_data
+      in [:sending_continuation_and_data, :send, FrameType::CONTINUATION] if frame.end_headers?
         :sending_data
-      in [:sending_continuation_data, FrameType::CONTINUATION, :send]
+      in [:sending_continuation_and_data, :send, FrameType::CONTINUATION]
         state
-      in [:sending_continuation_data, _, :send]
+      in [:sending_continuation_and_data, :send, _]
         unreachable(state, typ, direction)
-      in [:sending_continuation_data, _, :recv]
-        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
+      in [:sending_continuation_and_data, :recv, FrameType::PRIORITY]
+        state
+      in [:sending_continuation_and_data, :recv, FrameType::WINDOW_UPDATE]
+        state
+      in [:sending_continuation_and_data, _, FrameType::RST_STREAM]
+        :closed
 
       # sending_continuation
-      in [:sending_continuation, FrameType::RST_STREAM, :send]
+      in [:sending_continuation, :send, FrameType::CONTINUATION] if frame.end_headers?
         :closed
-      in [:sending_continuation, FrameType::WINDOW_UPDATE, :recv]
+      in [:sending_continuation, :send, FrameType::CONTINUATION]
         state
-      in [:sending_continuation, FrameType::CONTINUATION, :send] if frame.end_headers?
-        :closed
-      in [:sending_continuation, FrameType::CONTINUATION, :send]
-        state
-      in [:sending_continuation, _, :send]
+      in [:sending_continuation, :send, _]
         unreachable(state, typ, direction)
-      in [:sending_continuation, _, :recv]
-        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
+      in [:sending_continuation, :recv, FrameType::PRIORITY]
+        state
+      in [:sending_continuation, :recv, FrameType::WINDOW_UPDATE]
+        state
+      in [:sending_continuation, _, FrameType::RST_STREAM]
+        :closed
 
       # sending_data
-      in [:sending_data, FrameType::DATA, :send] if frame.end_stream?
+      in [:sending_data, :send, FrameType::DATA] if frame.end_stream?
         :closed
-      in [:sending_data, FrameType::WINDOW_UPDATE, :recv]
+      in [:sending_data, :send, FrameType::DATA]
+        state
+      in [:sending_data, :send, _]
+        unreachable(state, typ, direction)
+      in [:sending_data, :recv, FrameType::PRIORITY]
         state
-      in [:sending_data, FrameType::DATA, :send]
+      in [:sending_data, :recv, FrameType::WINDOW_UPDATE]
         state
-      in [:sending_data, FrameType::RST_STREAM, :send]
+      in [:sending_data, _, FrameType::RST_STREAM]
         :closed
-      in [:sending_continuation, _, :send]
-        unreachable(state, typ, direction)
-      in [:sending_continuation, _, :recv]
-        unexpected(ErrorCode::PROTOCOL_ERROR, state, typ, direction)
 
       # closed
-      in [:closed, FrameType::PRIORITY, :recv]
+      in [:closed, :send, _]
+        unreachable(state, typ, direction)
+      in [:closed, :recv, FrameType::PRIORITY]
         state
-      in [:closed, FrameType::RST_STREAM, :recv]
+      in [:closed, _, FrameType::RST_STREAM]
         state
-      in [:closed, _, :send]
-        unreachable(state, typ, direction)
-      in [:closed, _, :recv]
-        unexpected(ErrorCode::STREAM_CLOSED, state, typ, direction)
 
       # other
-      in [_, _, :send]
+      in [_, :send, _]
         unreachable(state, typ, direction)
-      in [_, _, :recv]
+      in [_, :recv, _]
         unexpected(ErrorCode::STREAM_CLOSED, state, typ, direction)
       end
     end

data/lib/biryani/streams_context.rb

@@ -47,9 +47,9 @@ module Biryani
       @h.values.filter(&:active?).length
     end
 
-    # @return [Boolean]
-    def receiving_continuation?
-      @h.values.any?(&:receiving_continuation?)
+    # @return [Integer, nil]
+    def receiving_continuation_stream_id
+      @h.find { |_, ctx| ctx.receiving_continuation? }&.first
     end
 
     # @return [Array<Integer>]
@@ -95,13 +95,13 @@ module Biryani
     def close_all
       each do |ctx|
         ctx.tx.close
-        ctx.state.close
+        ctx.close
       end
     end
   end
 
   class StreamContext
-    attr_accessor :stream, :tx, :send_window, :recv_window, :fragment, :content, :state
+    attr_accessor :tx, :send_window, :recv_window, :fragment, :content
 
     # @param stream_id [Integer]
     # @param send_initial_window_size [Integer]
@@ -117,6 +117,21 @@ module Biryani
       @state = State.new
     end
 
+    # @param req [HTTPRequest]
+    def <<(req)
+      @stream.rx.send(req, move: true)
+    end
+
+    def close
+      @state.close
+    end
+
+    # @param frame [Object]
+    # @param direction [:send, :recv]
+    def state_transition!(frame, direction)
+      @state.transition!(frame, direction)
+    end
+
     # @return [Boolean]
     def closed?
       @state.closed?

data/lib/biryani/version.rb

@@ -1,3 +1,3 @@
 module Biryani
-  VERSION = '0.0.6'.freeze
+  VERSION = '0.0.7'.freeze
 end

data/spec/connection/send_spec.rb

@@ -22,7 +22,7 @@ RSpec.describe Connection do
       streams_ctx
     end
     it 'should send' do
-      streams_ctx1[2].state.transition!(headers1, :recv)
+      streams_ctx1[2].state_transition!(headers1, :recv)
       Connection.send_headers(io, 2, "\x88".b, false, 16_384, streams_ctx1)
       Connection.send_data(io, 2, 'Hello, world!'.b, send_window1, 16_384, streams_ctx1, data_buffer)
       expect(io.string.force_encoding(Encoding::ASCII_8BIT)).to eq "\x00\x00\x01\x01\x04\x00\x00\x00\x02\x88\x00\x00\x0d\x00\x01\x00\x00\x00\x02Hello, world!".b
@@ -45,7 +45,7 @@ RSpec.describe Connection do
       streams_ctx
     end
     it 'should send' do
-      streams_ctx2[1].state.transition!(headers2, :recv)
+      streams_ctx2[1].state_transition!(headers2, :recv)
       Connection.send_headers(io, 1, "\x88".b, true, 16_384, streams_ctx2)
       expect(io.string.force_encoding(Encoding::ASCII_8BIT)).to eq "\x00\x00\x01\x01\x05\x00\x00\x00\x01\x88".b
       expect(send_window2.length).to eq 65_535
@@ -68,7 +68,7 @@ RSpec.describe Connection do
       streams_ctx
     end
     it 'should send' do
-      streams_ctx3[2].state.transition!(headers3, :recv)
+      streams_ctx3[2].state_transition!(headers3, :recv)
       Connection.send_data(io, 2, 'Hello, world!'.b, send_window3, 16_384, streams_ctx3, data_buffer)
       expect(io.string.force_encoding(Encoding::ASCII_8BIT)).to eq ''.b
       expect(send_window3.length).to eq 65_535
@@ -92,7 +92,7 @@ RSpec.describe Connection do
       streams_ctx
     end
     it 'should send' do
-      streams_ctx4[2].state.transition!(headers4, :recv)
+      streams_ctx4[2].state_transition!(headers4, :recv)
       Connection.send_data(io, 2, 'Hello, world!'.b, send_window4, 16_384, streams_ctx4, data_buffer)
       expect(io.string.force_encoding(Encoding::ASCII_8BIT)).to eq ''.b
       expect(send_window4.length).to eq 0

data/spec/connection/transition_stream_state_send_spec.rb

@@ -13,14 +13,14 @@ RSpec.describe Connection do
       Frame::Headers.new(true, true, 2, nil, nil, 'this is dummy', nil)
     end
     it 'should transition' do
-      streams_ctx[2].state.transition!(headers, :recv)
+      streams_ctx[2].state_transition!(headers, :recv)
       Connection.transition_stream_state_send(headers, streams_ctx)
       expect(streams_ctx.length).to eq 2
     end
 
     it 'should transition' do
-      streams_ctx[1].state.transition!(headers, :recv)
-      streams_ctx[2].state.transition!(headers, :recv)
+      streams_ctx[1].state_transition!(headers, :recv)
+      streams_ctx[2].state_transition!(headers, :recv)
       streams_ctx.close_all
       expect { streams_ctx[1].tx << nil }.to raise_error Ractor::ClosedError
       expect { streams_ctx[2].tx << nil }.to raise_error Ractor::ClosedError
@@ -30,7 +30,7 @@ RSpec.describe Connection do
       Frame::RstStream.new(2, 0)
     end
     it 'should transition' do
-      streams_ctx[2].state.transition!(headers, :recv)
+      streams_ctx[2].state_transition!(headers, :recv)
       Connection.transition_stream_state_send(rst_stream, streams_ctx)
       expect { streams_ctx[1].tx << nil }.to_not raise_error
       expect { streams_ctx[2].tx << nil }.to_not raise_error

data/spec/{http_request_builder_spec.rb → http_request_spec.rb}

@@ -1,5 +1,22 @@
 require_relative 'spec_helper'
 
+RSpec.describe HTTPRequest do
+  context 'trailers' do
+    it 'should be empty' do
+      expect(HTTPRequest.new('get', 'http://localhost:8888/', {}, '').trailers.empty?).to eq true
+    end
+
+    let(:req) do
+      HTTPRequest.new('get', 'http://localhost:8888/', { 'trailer' => %w[a b], 'a' => ['1'], 'b' => ['2'], 'c' => ['3'] }, '')
+    end
+    it 'should return' do
+      expect(req.trailers['a']).to eq ['1']
+      expect(req.trailers['b']).to eq ['2']
+      expect(req.trailers['c']).to eq nil
+    end
+  end
+end
+
 RSpec.describe HTTPRequestBuilder do
   context 'field' do
     let(:builder) do
@@ -39,7 +56,7 @@ RSpec.describe HTTPRequestBuilder do
       builder.build('')
     end
     it 'should field' do
-      expect(request.fields['cookie']).to eq 'a=1; b=2; c=3; d=4'
+      expect(request.fields['cookie']).to eq ['a=1; b=2; c=3; d=4']
     end
   end
 end

data/spec/streams_context_spec.rb

@@ -48,7 +48,7 @@ RSpec.describe StreamsContext do
       streams_ctx = StreamsContext.new(do_nothing_proc)
       streams_ctx.new_context(1, 65_535, 65_535)
       streams_ctx.new_context(2, 65_535, 65_535)
-      streams_ctx[2].state.close
+      streams_ctx[2].close
       streams_ctx
     end
     let(:data_buffer2) do
@@ -63,7 +63,7 @@ RSpec.describe StreamsContext do
       streams_ctx = StreamsContext.new(do_nothing_proc)
       streams_ctx.new_context(1, 65_535, 65_535)
       streams_ctx.new_context(2, 65_535, 65_535)
-      streams_ctx[2].state.close
+      streams_ctx[2].close
       streams_ctx
     end
     let(:data_buffer3) do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: biryani
 version: !ruby/object:Gem::Version
-  version: 0.0.6
+  version: 0.0.7
 platform: ruby
 authors:
 - thekuwayama
@@ -31,6 +31,7 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".github/workflows/ci.yml"
+- ".github/workflows/conformance.yml"
 - ".gitignore"
 - ".rubocop.yml"
 - ".ruby-version"
@@ -111,7 +112,7 @@ files:
 - spec/hpack/huffman_spec.rb
 - spec/hpack/integer_spec.rb
 - spec/hpack/string_spec.rb
-- spec/http_request_builder_spec.rb
+- spec/http_request_spec.rb
 - spec/http_response_spec.rb
 - spec/spec_helper.rb
 - spec/streams_context_spec.rb
@@ -167,7 +168,7 @@ test_files:
 - spec/hpack/huffman_spec.rb
 - spec/hpack/integer_spec.rb
 - spec/hpack/string_spec.rb
-- spec/http_request_builder_spec.rb
+- spec/http_request_spec.rb
 - spec/http_response_spec.rb
 - spec/spec_helper.rb
 - spec/streams_context_spec.rb