tipi 0.51 → 0.52

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6338c894cec0a59a270a7c75535441ab6be2c8a6fbb5457b8ecfd200ade25b92
-  data.tar.gz: 620303ca676af631574e678ca8e43f3af679d6c2dddbaeb29b1a1e3c3f3fe716
+  metadata.gz: efec664af01dfb4781c30670e2471a591a6c8df3a8024b5f14dc4edded3f5c1d
+  data.tar.gz: 0f384c5a305c5afe524b9a0e28d2a06931340275fe879e963cc194d2cde3c766
 SHA512:
-  metadata.gz: 2fc80e081f7f933ab2f2f352aae042e0cd396942b460c2061a6064b2ba90c31011c8e8632a3abebc17c8904b7406164b9771d2a0645b868a6de0d79b0a296c67
-  data.tar.gz: 295b6f82fc7a1bbc4e5b15e8c49732aa05d66b0b366ad93aa310c17b7be727a71f179f68358e5905d06efa29ad0b2f8a95d948f7625ea9455dac74b1b32f023e
+  metadata.gz: 7aa3995cd7ad560f3ba0f8a318da6c5ef62d53bc02209f05491f1e3db5b140e2697e8cd3158686a62c3dbae373d0260d803befabf4b8c417e862f2e6f68860bf
+  data.tar.gz: 7a17368c2fd9572531ac137822727082e48b85322190b91197f699d34d0554fd39d5dc7a18fe6f68bf8ea64dadbabbb464f39fa2b83475ee52cee99484c422a5

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.52 2022-03-03
+
+- Treat HTTP/2 headers as immutable
+
 ## 0.51 2022-02-28
 
 - Update dependencies

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    tipi (0.51)
+    tipi (0.52)
       acme-client (~> 2.0.9)
       ever (~> 0.1)
       extralite (~> 1.14)
@@ -56,7 +56,7 @@ GEM
     minitest (5.11.3)
     msgpack (1.4.5)
     multipart-post (2.1.1)
-    polyphony (0.80)
+    polyphony (0.81)
     qeweney (0.18)
       escape_utils (~> 1.2.1)
     rack (2.2.3)

data/lib/tipi/http2_stream.rb

@@ -113,21 +113,19 @@ module Tipi
     end
 
     # response API
-    def respond(request, chunk, headers)
-      headers[':status'] ||= Qeweney::Status::OK
-      headers[':status'] = headers[':status'].to_s
+    def respond(request, body, headers)
+      headers = normalize_status_header(headers)
       with_transfer_count(request) do
         @stream.headers(transform_headers(headers))
         @headers_sent = true
-        @stream.data(chunk || '')
+        @stream.data(body || '')
       end
     rescue HTTP2::Error::StreamClosed
       # ignore
     end
 
     def respond_from_io(request, io, headers, chunk_size = 2**16)
-      headers[':status'] ||= Qeweney::Status::OK
-      headers[':status'] = headers[':status'].to_s
+      headers = normalize_status_header(headers)
       with_transfer_count(request) do
         @stream.headers(transform_headers(headers))
         @headers_sent = true
@@ -152,7 +150,8 @@ module Tipi
     def send_headers(request, headers, empty_response: false)
       return if @headers_sent
 
-      headers[':status'] ||= (empty_response ? Qeweney::Status::NO_CONTENT : Qeweney::Status::OK).to_s
+      status = empty_response ? Qeweney::Status::NO_CONTENT : Qeweney::Status::OK
+      headers = normalize_status_header(headers, status)
       with_transfer_count(request) do
         @stream.headers(transform_headers(headers), end_stream: false)
       end
@@ -194,5 +193,17 @@ module Tipi
       @stream.close
       @stream_fiber.schedule(Polyphony::MoveOn.new)
     end
+
+    private
+
+    def normalize_status_header(headers, default_status = Qeweney::Status::OK)
+      if !headers[':status']
+        headers.merge(':status' => default_status.to_s)
+      elsif !headers[':status'].is_a?(String)
+        headers.merge(headers[':status'].to_s)
+      else
+        headers
+      end
+    end
   end
 end

data/lib/tipi/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Tipi
-  VERSION = '0.51'
+  VERSION = '0.52'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tipi
 version: !ruby/object:Gem::Version
-  version: '0.51'
+  version: '0.52'
 platform: ruby
 authors:
 - Sharon Rosner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-02-28 00:00:00.000000000 Z
+date: 2022-03-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: polyphony