http-2 0.8.2 → 0.8.3

data/lib/http/2/framer.rb

@@ -1,6 +1,7 @@
 module HTTP2
   # Performs encoding, decoding, and validation of binary HTTP/2 frames.
   #
+  # rubocop:disable ClassLength
   class Framer
     include Error
 
@@ -215,7 +216,7 @@ module HTTP2
         length += 4
 
       when :settings
-        if frame[:stream] != 0
+        if (frame[:stream]).nonzero?
           fail CompressionError, "Invalid stream ID (#{frame[:stream]})"
         end
 
@@ -268,7 +269,8 @@ module HTTP2
         length += 6
         if frame[:proto]
           fail CompressionError, 'Proto too long' if frame[:proto].bytesize > 255
-          bytes << [frame[:proto].bytesize].pack(UINT8) << frame[:proto].force_encoding(Encoding::BINARY)
+          bytes << [frame[:proto].bytesize].pack(UINT8)
+          bytes << frame[:proto].force_encoding(Encoding::BINARY)
           length += 1 + frame[:proto].bytesize
         else
           bytes << [0].pack(UINT8)
@@ -276,7 +278,8 @@ module HTTP2
         end
         if frame[:host]
           fail CompressionError, 'Host too long' if frame[:host].bytesize > 255
-          bytes << [frame[:host].bytesize].pack(UINT8) << frame[:host].force_encoding(Encoding::BINARY)
+          bytes << [frame[:host].bytesize].pack(UINT8)
+          bytes << frame[:host].force_encoding(Encoding::BINARY)
           length += 1 + frame[:host].bytesize
         else
           bytes << [0].pack(UINT8)
@@ -325,6 +328,8 @@ module HTTP2
       frame = read_common_header(buf)
       return nil if buf.size < 9 + frame[:length]
 
+      fail ProtocolError, 'payload too large' if frame[:length] > DEFAULT_MAX_FRAME_SIZE
+
       buf.read(9)
       payload = buf.read(frame[:length])
 
@@ -354,14 +359,14 @@ module HTTP2
         if frame[:flags].include? :priority
           e_sd = payload.read_uint32
           frame[:stream_dependency] = e_sd & RBIT
-          frame[:exclusive] = (e_sd & EBIT) != 0
+          frame[:exclusive] = (e_sd & EBIT) != 0 # rubocop:disable Style/NumericPredicate
           frame[:weight] = payload.getbyte + 1
         end
         frame[:payload] = payload.read(frame[:length])
       when :priority
         e_sd = payload.read_uint32
         frame[:stream_dependency] = e_sd & RBIT
-        frame[:exclusive] = (e_sd & EBIT) != 0
+        frame[:exclusive] = (e_sd & EBIT) != 0 # rubocop:disable Style/NumericPredicate
         frame[:weight] = payload.getbyte + 1
       when :rst_stream
         frame[:error] = unpack_error payload.read_uint32
@@ -370,11 +375,11 @@ module HTTP2
         # NOTE: frame[:length] might not match the number of frame[:payload]
         # because unknown extensions are ignored.
         frame[:payload] = []
-        unless frame[:length] % 6 == 0
+        unless (frame[:length] % 6).zero?
           fail ProtocolError, 'Invalid settings payload length'
         end
 
-        if frame[:stream] != 0
+        if (frame[:stream]).nonzero?
           fail ProtocolError, "Invalid stream ID (#{frame[:stream]})"
         end
 

data/lib/http/2/server.rb

@@ -1,3 +1,4 @@
+require 'base64'
 module HTTP2
   # HTTP 2.0 server connection class that implements appropriate header
   # compression / decompression algorithms and stream management logic.
@@ -71,7 +72,7 @@ module HTTP2
       receive(CONNECTION_PREFACE_MAGIC)
 
       # Process received HTTP2-Settings payload
-      buf = HTTP2::Buffer.new Base64.urlsafe_decode64(settings)
+      buf = HTTP2::Buffer.new Base64.urlsafe_decode64(settings.to_s)
       header = @framer.common_header(
         length: buf.bytesize,
         type: :settings,

data/lib/http/2/stream.rb

@@ -54,7 +54,7 @@ module HTTP2
     # Size of current stream flow control window.
     attr_reader :local_window
     attr_reader :remote_window
-    alias_method :window, :local_window
+    alias window local_window
 
     # Reason why connection was closed.
     attr_reader :closed
@@ -117,7 +117,7 @@ module HTTP2
 
       complete_transition(frame)
     end
-    alias_method :<<, :receive
+    alias << receive
 
     # Processes outgoing HTTP 2.0 frames. Data frames may be automatically
     # split and buffered based on maximum frame size and current stream flow
@@ -182,9 +182,11 @@ module HTTP2
       # Split data according to each frame is smaller enough
       # TODO: consider padding?
       max_size = @connection.remote_settings[:settings_max_frame_size]
-      while payload.bytesize > max_size
-        chunk = payload.slice!(0, max_size)
-        send(type: :data, flags: [], payload: chunk)
+
+      if payload.bytesize > max_size
+        payload = chunk_data(payload, max_size) do |chunk|
+          send(type: :data, flags: [], payload: chunk)
+        end
       end
 
       flags = []
@@ -192,6 +194,18 @@ module HTTP2
       send(type: :data, flags: flags, payload: payload)
     end
 
+    # Chunk data into max_size, yield each chunk, then return final chunk
+    #
+    def chunk_data(payload, max_size)
+      total = payload.bytesize
+      cursor = 0
+      while (total - cursor) > max_size
+        yield payload.byteslice(cursor, max_size)
+        cursor += max_size
+      end
+      payload.byteslice(cursor, total - cursor)
+    end
+
     # Sends a RST_STREAM frame which closes current stream - this does not
     # close the underlying connection.
     #
@@ -320,14 +334,14 @@ module HTTP2
       # WINDOW_UPDATE on a stream in this state MUST be treated as a
       # connection error (Section 5.4.1) of type PROTOCOL_ERROR.
       when :reserved_local
-        if sending
-          @state = case frame[:type]
+        @state = if sending
+          case frame[:type]
           when :headers     then event(:half_closed_remote)
           when :rst_stream  then event(:local_rst)
           else stream_error
           end
         else
-          @state = case frame[:type]
+          case frame[:type]
           when :rst_stream then event(:remote_rst)
           when :priority, :window_update then @state
           else stream_error
@@ -349,14 +363,14 @@ module HTTP2
       # PRIORITY on a stream in this state MUST be treated as a connection
       # error (Section 5.4.1) of type PROTOCOL_ERROR.
       when :reserved_remote
-        if sending
-          @state = case frame[:type]
+        @state = if sending
+          case frame[:type]
           when :rst_stream then event(:local_rst)
           when :priority, :window_update then @state
           else stream_error
           end
         else
-          @state = case frame[:type]
+          case frame[:type]
           when :headers then event(:half_closed_local)
           when :rst_stream then event(:remote_rst)
           else stream_error

data/lib/http/2/version.rb

@@ -1,3 +1,3 @@
 module HTTP2
-  VERSION = '0.8.2'.freeze
+  VERSION = '0.8.3'.freeze
 end

data/lib/tasks/generate_huffman_table.rb

@@ -25,10 +25,10 @@ module HuffmanTable
 
     def add(code, len, chr)
       self.final = true if chr == EOS && @depth <= 7
-      if len == 0
+      if len.zero?
         @emit = chr
       else
-        bit = (code & (1 << (len - 1))) == 0 ? 0 : 1
+        bit = (code & (1 << (len - 1))).zero? ? 0 : 1
         node = @next[bit] ||= Node.new(@depth + 1)
         node.add(code, len - 1, chr)
       end
@@ -68,7 +68,7 @@ module HuffmanTable
           n = node
           emit = ''
           (BITS_AT_ONCE - 1).downto(0) do |i|
-            bit = (input & (1 << i)) == 0 ? 0 : 1
+            bit = (input & (1 << i)).zero? ? 0 : 1
             n = n.next[bit]
             next unless n.emit
             if n.emit == EOS

data/spec/client_spec.rb

@@ -56,10 +56,10 @@ RSpec.describe HTTP2::Client do
     it 'should raise error on PUSH_PROMISE against non-idle stream' do
       expect do
         s = @client.new_stream
-        s.send HEADERS
+        s.send HEADERS.dup
 
-        @client << set_stream_id(f.generate(PUSH_PROMISE), s.id)
-        @client << set_stream_id(f.generate(PUSH_PROMISE), s.id)
+        @client << set_stream_id(f.generate(PUSH_PROMISE.dup), s.id)
+        @client << set_stream_id(f.generate(PUSH_PROMISE.dup), s.id)
       end.to raise_error(ProtocolError)
     end
 

data/spec/connection_spec.rb

@@ -149,7 +149,6 @@ RSpec.describe HTTP2::Connection do
       ]
       headers = []
       @conn.on(:frame) do |bytes|
-        # bytes[3]: frame's type field
         headers << f.parse(bytes) if [1, 5, 9].include?(bytes[3].ord)
       end
 

data/spec/h2spec/output/non_secure.txt

@@ -0,0 +1,317 @@
+  3.5. HTTP/2 Connection Preface
+      Sends invalid connection preface
    ✓ Sends invalid connection preface
+
+  4.2. Frame Size
+      Sends large size frame that exceeds the SETTINGS_MAX_FRAME_SIZE
    ✓ Sends large size frame that exceeds the SETTINGS_MAX_FRAME_SIZE
+
+  4.3. Header Compression and Decompression
+      Sends invalid header block fragment
    ✓ Sends invalid header block fragment
+      Sends Dynamic Table Size Update (RFC 7541, 6.3)
    ✓ Sends Dynamic Table Size Update (RFC 7541, 6.3)
+      Encodes Dynamic Table Size Update (RFC 7541, 6.3) after common header fields
    × Encodes Dynamic Table Size Update (RFC 7541, 6.3) after common header fields
+      - The endpoint MUST terminate the connection with a connection error of type COMPRESSION_ERROR.
+        Expected: GOAWAY frame (ErrorCode: COMPRESSION_ERROR)
+                  Connection close
+          Actual: DATA frame (Length: 22, Flags: 1)
+
+  5.1. Stream States
+      idle: Sends a DATA frame
    ✓ idle: Sends a DATA frame
+      idle: Sends a RST_STREAM frame
    ✓ idle: Sends a RST_STREAM frame
+      idle: Sends a WINDOW_UPDATE frame
    ✓ idle: Sends a WINDOW_UPDATE frame
+      idle: Sends a CONTINUATION frame
    ✓ idle: Sends a CONTINUATION frame
+      half closed (remote): Sends a DATA frame
    × half closed (remote): Sends a DATA frame
+      - The endpoint MUST respond with a stream error (Section 5.4.2) of type STREAM_CLOSED.
+        Expected: GOAWAY frame (ErrorCode: STREAM_CLOSED)
+                  RST_STREAM frame (ErrorCode: STREAM_CLOSED)
+                  Connection close
+          Actual: WINDOW_UPDATE frame (Length: 4, Flags: 0)
+      half closed (remote): Sends a HEADERS frame
    × half closed (remote): Sends a HEADERS frame
+      - The endpoint MUST respond with a stream error (Section 5.4.2) of type STREAM_CLOSED.
+        Expected: GOAWAY frame (ErrorCode: STREAM_CLOSED)
+                  RST_STREAM frame (ErrorCode: STREAM_CLOSED)
+                  Connection close
+          Actual: DATA frame (Length: 22, Flags: 1)
+      half closed (remote): Sends a CONTINUATION frame
    × half closed (remote): Sends a CONTINUATION frame
+      - The endpoint MUST respond with a stream error (Section 5.4.2) of type STREAM_CLOSED.
+        Expected: GOAWAY frame (ErrorCode: STREAM_CLOSED)
+                  RST_STREAM frame (ErrorCode: STREAM_CLOSED)
+                  GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: DATA frame (Length: 22, Flags: 1)
+      closed: Sends a CONTINUATION frame
    × closed: Sends a CONTINUATION frame
+      - The endpoint MUST treat this as a stream error (Section 5.4.2) of type STREAM_CLOSED.
+        Expected: GOAWAY frame (ErrorCode: STREAM_CLOSED)
+                  RST_STREAM frame (ErrorCode: STREAM_CLOSED)
+                  GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: Test timeout
+
+    5.1.1. Stream Identifiers
+        Sends even-numbered stream identifier
      ✓ Sends even-numbered stream identifier
+
+    5.1.2. Stream Concurrency
+        Sends HEADERS frames that causes their advertised concurrent stream limit to be exceeded
      × Sends HEADERS frames that causes their advertised concurrent stream limit to be exceeded
+        - The endpoint MUST treat this as a stream error (Section 5.4.2) of type PROTOCOL_ERROR or REFUSED_STREAM
+          Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                    RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                    GOAWAY frame (ErrorCode: REFUSED_STREAM)
+                    RST_STREAM frame (ErrorCode: REFUSED_STREAM)
+                    Connection close
+            Actual: HEADERS frame (Length: 3, Flags: 4)
+
+  5.3. Stream Priority
+    5.3.1. Stream Dependencies
+        Sends HEADERS frame that depend on itself
      × Sends HEADERS frame that depend on itself
+        - The endpoint MUST treat this as a stream error of type PROTOCOL_ERROR
+          Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                    RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                    Connection close
+            Actual: DATA frame (Length: 22, Flags: 1)
+        Sends PRIORITY frame that depend on itself
      × Sends PRIORITY frame that depend on itself
+        - The endpoint MUST treat this as a stream error of type PROTOCOL_ERROR
+          Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                    RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                    Connection close
+            Actual: Test timeout
+
+  5.5. Extending HTTP/2
+      Sends an unknown extension frame
    ✓ Sends an unknown extension frame
+      Sends an unknown extension frame in the middle of a header block
    × Sends an unknown extension frame in the middle of a header block
+      - The endpoint MUST treat as a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: Test timeout
+
+  6.1. DATA
+      Sends a DATA frame with 0x0 stream identifier
    ✓ Sends a DATA frame with 0x0 stream identifier
+      Sends a DATA frame on the stream that is not in "open" or "half-closed (local)" state
    × Sends a DATA frame on the stream that is not in "open" or "half-closed (local)" state
+      - The endpoint MUST respond with a stream error (Section 5.4.2) of type STREAM_CLOSED.
+        Expected: GOAWAY frame (ErrorCode: STREAM_CLOSED)
+                  RST_STREAM frame (ErrorCode: STREAM_CLOSED)
+                  Connection close
+          Actual: WINDOW_UPDATE frame (Length: 4, Flags: 0)
+      Sends a DATA frame with invalid pad length
    ✓ Sends a DATA frame with invalid pad length
+
+  6.2. HEADERS
+      Sends a HEADERS frame followed by any frame other than CONTINUATION
    ✓ Sends a HEADERS frame followed by any frame other than CONTINUATION
+      Sends a HEADERS frame followed by a frame on a different stream
    × Sends a HEADERS frame followed by a frame on a different stream
+      - The endpoint MUST treat the receipt of a frame on a different stream as a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: Test timeout
+      Sends a HEADERS frame with 0x0 stream identifier
    ✓ Sends a HEADERS frame with 0x0 stream identifier
+      Sends a HEADERS frame with invalid pad length
    ✓ Sends a HEADERS frame with invalid pad length
+
+  6.3. PRIORITY
+      Sends a PRIORITY frame with 0x0 stream identifier
    ✓ Sends a PRIORITY frame with 0x0 stream identifier
+      Sends a PRIORITY frame with a length other than 5 octets
    × Sends a PRIORITY frame with a length other than 5 octets
+      - The endpoint MUST respond with a stream error of type FRAME_SIZE_ERROR.
+        Expected: GOAWAY frame (ErrorCode: FRAME_SIZE_ERROR)
+                  RST_STREAM frame (ErrorCode: FRAME_SIZE_ERROR)
+                  Connection close
+          Actual: GOAWAY frame (Length: 8, Flags: 0, ErrorCode: PROTOCOL_ERROR)
+
+  6.4. RST_STREAM
+      Sends a RST_STREAM frame with 0x0 stream identifier
    ✓ Sends a RST_STREAM frame with 0x0 stream identifier
+      Sends a RST_STREAM frame on a idle stream
    ✓ Sends a RST_STREAM frame on a idle stream
+      Sends a RST_STREAM frame with a length other than 4 octets
    × Sends a RST_STREAM frame with a length other than 4 octets
+      - The endpoint MUST respond with a connection error of type FRAME_SIZE_ERROR.
+        Expected: GOAWAY frame (ErrorCode: FRAME_SIZE_ERROR)
+                  Connection close
+          Actual: GOAWAY frame (Length: 8, Flags: 0, ErrorCode: PROTOCOL_ERROR)
+
+  6.5. SETTINGS
+      Sends a SETTINGS frame
    ✓ Sends a SETTINGS frame
+      Sends a SETTINGS frame that is not a zero-length with ACK flag
    ✓ Sends a SETTINGS frame that is not a zero-length with ACK flag
+      Sends a SETTINGS frame with the stream identifier that is not 0x0
    ✓ Sends a SETTINGS frame with the stream identifier that is not 0x0
+      Sends a SETTINGS frame with a length other than a multiple of 6 octets
    ✓ Sends a SETTINGS frame with a length other than a multiple of 6 octets
+
+    6.5.2. Defined SETTINGS Parameters
+        SETTINGS_ENABLE_PUSH (0x2): Sends the value other than 0 or 1
      ✓ SETTINGS_ENABLE_PUSH (0x2): Sends the value other than 0 or 1
+        SETTINGS_INITIAL_WINDOW_SIZE (0x4): Sends the value above the maximum flow control window size
      × SETTINGS_INITIAL_WINDOW_SIZE (0x4): Sends the value above the maximum flow control window size
+        - The endpoint MUST respond with a connection error of type FLOW_CONTROL_ERROR.
+          Expected: GOAWAY frame (ErrorCode: FLOW_CONTROL_ERROR)
+                    Connection close
+            Actual: GOAWAY frame (Length: 8, Flags: 0, ErrorCode: PROTOCOL_ERROR)
+        SETTINGS_MAX_FRAME_SIZE (0x5): Sends the value below the initial value
      ✓ SETTINGS_MAX_FRAME_SIZE (0x5): Sends the value below the initial value
+        SETTINGS_MAX_FRAME_SIZE (0x5): Sends the value above the maximum allowed frame size
      ✓ SETTINGS_MAX_FRAME_SIZE (0x5): Sends the value above the maximum allowed frame size
+
+  6.7. PING
+      Sends a PING frame
    ✓ Sends a PING frame
+      Sends a PING frame with the stream identifier that is not 0x0
    × Sends a PING frame with the stream identifier that is not 0x0
+      - The endpoint MUST respond with a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: PING frame (Length: 8, Flags: 1)
+      Sends a PING frame with a length field value other than 8
    ✓ Sends a PING frame with a length field value other than 8
+
+  6.8. GOAWAY
+      Sends a GOAWAY frame with the stream identifier that is not 0x0
    × Sends a GOAWAY frame with the stream identifier that is not 0x0
+      - The endpoint MUST respond with a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: Test timeout
+
+  6.9. WINDOW_UPDATE
+      Sends a WINDOW_UPDATE frame
    × Sends a WINDOW_UPDATE frame
+      - The endpoint is expected to send the DATA frame based on the window size.
+        Expected: DATA frame
+          Actual: Test timeout
+      Sends a WINDOW_UPDATE frame with a flow control window increment of 0
    × Sends a WINDOW_UPDATE frame with a flow control window increment of 0
+      - The endpoint MUST respond with a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: Test timeout
+      Sends a WINDOW_UPDATE frame with a flow control window increment of 0 on a stream
    × Sends a WINDOW_UPDATE frame with a flow control window increment of 0 on a stream
+      - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: SETTINGS frame (Length: 0, Flags: 1)
+      Sends a WINDOW_UPDATE frame with a length other than a multiple of 4 octets
    × Sends a WINDOW_UPDATE frame with a length other than a multiple of 4 octets
+      - The endpoint MUST respond with a connection error of type FRAME_SIZE_ERROR.
+        Expected: GOAWAY frame (ErrorCode: FRAME_SIZE_ERROR)
+                  Connection close
+          Actual: GOAWAY frame (Length: 8, Flags: 0, ErrorCode: PROTOCOL_ERROR)
+
+    6.9.1. The Flow Control Window
+        Sends multiple WINDOW_UPDATE frames on a connection increasing the flow control window to above 2^31-1
      × Sends multiple WINDOW_UPDATE frames on a connection increasing the flow control window to above 2^31-1
+        - The endpoint MUST sends a GOAWAY frame with a FLOW_CONTROL_ERROR code.
+          Expected: GOAWAY frame (ErrorCode: FLOW_CONTROL_ERROR)
+            Actual: Test timeout
+        Sends multiple WINDOW_UPDATE frames on a stream increasing the flow control window to above 2^31-1
      × Sends multiple WINDOW_UPDATE frames on a stream increasing the flow control window to above 2^31-1
+        - The endpoint MUST send a RST_STREAM with the error code of FLOW_CONTROL_ERROR code.
+          Expected: RST_STREAM frame (ErrorCode: FLOW_CONTROL_ERROR)
+                    GOAWAY frame (ErrorCode: FLOW_CONTROL_ERROR)
+            Actual: Test timeout
+
+    6.9.2. Initial Flow Control Window Size
+        Sends a SETTINGS_INITIAL_WINDOW_SIZE settings with an exceeded maximum window size value
      × Sends a SETTINGS_INITIAL_WINDOW_SIZE settings with an exceeded maximum window size value
+        - The endpoint MUST respond with a connection error of type FLOW_CONTROL_ERROR.
+          Expected: GOAWAY frame (ErrorCode: FLOW_CONTROL_ERROR)
+                    Connection close
+            Actual: GOAWAY frame (Length: 8, Flags: 0, ErrorCode: PROTOCOL_ERROR)
+
+  6.10. CONTINUATION
+      Sends a CONTINUATION frame
    ✓ Sends a CONTINUATION frame
+      Sends multiple CONTINUATION frames
    ✓ Sends multiple CONTINUATION frames
+      Sends a CONTINUATION frame followed by any frame other than CONTINUATION
    × Sends a CONTINUATION frame followed by any frame other than CONTINUATION
+      - The endpoint MUST treat as a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: Test timeout
+      Sends a CONTINUATION frame followed by a frame on a different stream
    ✓ Sends a CONTINUATION frame followed by a frame on a different stream
+      Sends a CONTINUATION frame with the stream identifier that is 0x0
    ✓ Sends a CONTINUATION frame with the stream identifier that is 0x0
+      Sends a CONTINUATION frame after the frame other than HEADERS, PUSH_PROMISE or CONTINUATION
    × Sends a CONTINUATION frame after the frame other than HEADERS, PUSH_PROMISE or CONTINUATION
+      - The endpoint MUST treat as a connection error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: WINDOW_UPDATE frame (Length: 4, Flags: 0)
+
+  8.1. HTTP Request/Response Exchange
+      Sends a HEADERS frame as HEAD request
    × Sends a HEADERS frame as HEAD request
+      - The endpoint should respond with no DATA frame or empty DATA frame.
+        Expected: HEADERS frame (Flags: 1)
+                  DATA frame (Length: 0, Flags: 1)
+          Actual: DATA frame (Length: 22, Flags: 1)
+      Sends a HEADERS frame containing trailer part
    ✓ Sends a HEADERS frame containing trailer part
+      Sends a second HEADERS frame without the END_STREAM flag
    × Sends a second HEADERS frame without the END_STREAM flag
+      - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+        Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                  RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                  Connection close
+          Actual: WINDOW_UPDATE frame (Length: 4, Flags: 0)
+
+    8.1.2. HTTP Header Fields
+        Sends a HEADERS frame that contains the header field name in uppercase letters
      × Sends a HEADERS frame that contains the header field name in uppercase letters
+        - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+          Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                    RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                    Connection close
+            Actual: DATA frame (Length: 22, Flags: 1)
+
+      8.1.2.1. Pseudo-Header Fields
+          Sends a HEADERS frame that contains the pseudo-header field defined for response
        × Sends a HEADERS frame that contains the pseudo-header field defined for response
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame that contains the invalid pseudo-header field
        × Sends a HEADERS frame that contains the invalid pseudo-header field
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame that contains a pseudo-header field that appears in a header block after a regular header field
        × Sends a HEADERS frame that contains a pseudo-header field that appears in a header block after a regular header field
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+
+      8.1.2.2. Connection-Specific Header Fields
+          Sends a HEADERS frame that contains the connection-specific header field
        × Sends a HEADERS frame that contains the connection-specific header field
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame that contains the TE header field that contain any value other than "trailers"
        × Sends a HEADERS frame that contains the TE header field that contain any value other than "trailers"
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+
+      8.1.2.3. Request Pseudo-Header Fields
+          Sends a HEADERS frame that omits mandatory pseudo-header fields
        × Sends a HEADERS frame that omits mandatory pseudo-header fields
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame that omits just ':method' pseudo-header field.
        × Sends a HEADERS frame that omits just ':method' pseudo-header field.
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame that omits just ':scheme' pseudo-header field.
        × Sends a HEADERS frame that omits just ':scheme' pseudo-header field.
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame that omits just ':path' pseudo-header field.
        × Sends a HEADERS frame that omits just ':path' pseudo-header field.
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+          Sends a HEADERS frame containing more than one pseudo-header fields with the same name
        × Sends a HEADERS frame containing more than one pseudo-header fields with the same name
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 22, Flags: 1)
+
+      8.1.2.6. Malformed Requests and Responses
+          Sends a HEADERS frame that contains the "content-length" header field which does not equal the sum of the DATA frame payload lengths
        × Sends a HEADERS frame that contains the "content-length" header field which does not equal the sum of the DATA frame payload lengths
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 29, Flags: 1)
+          Sends a HEADERS frame that contains the "content-length" header field which does not equal the sum of the multiple DATA frame payload lengths
        × Sends a HEADERS frame that contains the "content-length" header field which does not equal the sum of the multiple DATA frame payload lengths
+          - The endpoint MUST respond with a stream error of type PROTOCOL_ERROR.
+            Expected: GOAWAY frame (ErrorCode: PROTOCOL_ERROR)
+                      RST_STREAM frame (ErrorCode: PROTOCOL_ERROR)
+                      Connection close
+              Actual: DATA frame (Length: 33, Flags: 1)
+
+  8.2. Server Push
+      Sends a PUSH_PROMISE frame
    ✓ Sends a PUSH_PROMISE frame
+
+73 tests, 33 passed, 0 skipped, 40 failed