h1p 0.6 → 0.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 90f7937fd6bcefdd1627c208bf73054fb3fb03a855666c7247f04aac4398d280
-  data.tar.gz: ab6a866b843beaf92137b991048db9af006f4cb5e510e211dc6ba6652bbc84a2
+  metadata.gz: 96197142deeb30d27fe9b3fc9de21a07ed1de8f0a1d3db9b8d967310056bdd12
+  data.tar.gz: aa71893e95f7a7e8ba8de8123987445262edd6dbcad7efba12337091a2421d14
 SHA512:
-  metadata.gz: 101e6e13aeed0cf2250dbe962c0c9c9edc5bf69e6ff2a59ee0d02018992c9b76dcaa4be3ddb44bbcb2272e355278a0b717c51050a2786cc95c00fbdb23bc40fa
-  data.tar.gz: 2d3130c067d51f173873d6181b76999dd897c241dcafc5fa3f69ec0837404b2fd7e292cadc2fbf9e27003e3dfd933dc394799abf03f12187d6c80e4f2dab83d6
+  metadata.gz: 9380e218a2433f61a88676fb807ecb742391dff2c6d7e4937d24ff1385839ed897eaf81eb3d8d1f916bcf53f30cd2fd9c31f0c78fcd6c0e35d3753dc1d3a1252
+  data.tar.gz: 2ce69a5e542390018f779e9ba0a73263b362b3a6d69838494da73d18cb6f84a3262ed98fd086bb098ffa253f264db4d3e1d40e5ae8f9757ca1928c36ce9f02ce

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 0.6.1 2023-05-28
+
+- Fix sending response with frozen headers hash
+
 ## 0.6 2023-01-05
 
 - Add documentation

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    h1p (0.6)
+    h1p (0.6.1)
 
 GEM
   remote: https://rubygems.org/

data/README.md

@@ -230,15 +230,22 @@ arbitrary IO instances. To write a response with or without a body, use
 
 ```ruby
 H1P.send_response(socket, { 'Some-Header' => 'header value'}, 'foobar')
-#=> "HTTP/1.1 200 OK\r\nSome-Header: header value\r\n\r\nfoobar"
+# HTTP/1.1 200 OK
+# Some-Header: header value
+# 
+# foobar
 
 # The :protocol pseudo header sets the protocol in the status line:
 H1P.send_response(socket, { ':protocol' => 'HTTP/0.9' })
-#=> "HTTP/0.9 200 OK\r\n\r\n"
+# HTTP/0.9 200 OK
+#
+#
 
 # The :status pseudo header sets the response status:
 H1P.send_response(socket, { ':status' => '418 I\'m a teapot' })
-#=> "HTTP/1.1 418 I'm a teapot\r\n\r\n"
+# HTTP/1.1 418 I'm a teapot
+#
+#
 ```
 
 To send responses using chunked transfer encoding use
@@ -246,7 +253,13 @@ To send responses using chunked transfer encoding use
 
 ```ruby
 H1P.send_chunked_response(socket, {}, "foobar")
-#=> "HTTP/1.1 200 OK\r\nTransfer-Encoding: chunked\r\n\r\n6\r\nfoobar\r\n0\r\n\r\n"
+# HTTP/1.1 200 OK
+# Transfer-Encoding: chunked
+# 6
+# foobar
+# 0
+#
+#
 ```
 
 You can also call `H1P.send_chunked_response` with a block that provides the
@@ -263,10 +276,14 @@ To send individual chunks use `H1P.send_body_chunk`:
 
 ```ruby
 H1P.send_body_chunk(socket, 'foo')
-#=> "3\r\nfoo\r\n"
+# 3
+# foo
+#
 
 H1P.send_body_chunk(socket, nil)
-#=> "0\r\n\r\n"
+# 0
+#
+#
 ```
 
 ## Parser Design

data/ext/h1p/h1p.c

@@ -1236,10 +1236,11 @@ VALUE H1P_send_response(int argc,VALUE *argv, VALUE self) {
 
     bodyptr = RSTRING_PTR(body);
     bodylen = RSTRING_LEN(body);
-    rb_hash_aset(headers, STR_content_length_capitalized, INT2FIX(bodylen));
+    // rb_hash_aset(headers, STR_content_length_capitalized, INT2FIX(bodylen));
   }
 
   rb_hash_foreach(headers, send_response_write_header, (VALUE)&ctx);
+  send_response_write_header(STR_content_length_capitalized, INT2FIX(bodylen), (VALUE)&ctx);
 
   char *endptr = ctx.buffer_ptr + ctx.buffer_len;
   endptr[0] = '\r';
@@ -1309,8 +1310,8 @@ VALUE H1P_send_chunked_response(VALUE self, VALUE io, VALUE headers) {
   if (status == Qnil) status = STR_pseudo_status_default;
   send_response_write_status_line(&ctx, protocol, status);
 
-  rb_hash_aset(headers, STR_transfer_encoding_capitalized, STR_chunked);
   rb_hash_foreach(headers, send_response_write_header, (VALUE)&ctx);
+  send_response_write_header(STR_transfer_encoding_capitalized, STR_chunked, (VALUE)&ctx);
 
   ctx.buffer_ptr[ctx.buffer_len] = '\r';
   ctx.buffer_ptr[ctx.buffer_len + 1] = '\n';

data/lib/h1p/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module H1P
-  VERSION = '0.6'
+  VERSION = '0.6.1'
 end

data/test/test_h1p.rb

@@ -9,30 +9,30 @@ class SendResponseTest < MiniTest::Test
     H1P.send_response(o, { ':status' => '418 I\'m a teapot' })
     o.close
     response = i.read
-    assert_equal "HTTP/1.1 418 I'm a teapot\r\n\r\n", response
+    assert_equal "HTTP/1.1 418 I'm a teapot\r\nContent-Length: 0\r\n\r\n", response
 
     i, o = IO.pipe
     count = H1P.send_response(o, { ':protocol' => 'HTTP/1.0' })
     o.close
     response = i.read
-    assert_equal "HTTP/1.0 200 OK\r\n\r\n", response
-    assert_equal "HTTP/1.0 200 OK\r\n\r\n".bytesize, count
+    assert_equal "HTTP/1.0 200 OK\r\nContent-Length: 0\r\n\r\n", response
+    assert_equal "HTTP/1.0 200 OK\r\nContent-Length: 0\r\n\r\n".bytesize, count
   end
 
   def test_send_response_string_headers
     i, o = IO.pipe
-    H1P.send_response(o, { 'Foo' => 'Bar', 'Content-Length' => '123' })
+    H1P.send_response(o, { 'Foo' => 'Bar', 'X-Blah' => '123' })
     o.close
     response = i.read
-    assert_equal "HTTP/1.1 200 OK\r\nFoo: Bar\r\nContent-Length: 123\r\n\r\n", response
+    assert_equal "HTTP/1.1 200 OK\r\nFoo: Bar\r\nX-Blah: 123\r\nContent-Length: 0\r\n\r\n", response
   end
 
   def test_send_response_non_string_headers
     i, o = IO.pipe
-    H1P.send_response(o, { :Foo => 'Bar', 'Content-Length' => 123 })
+    H1P.send_response(o, { :Foo => 'Bar', 'X-Blah' => 123 })
     o.close
     response = i.read
-    assert_equal "HTTP/1.1 200 OK\r\nFoo: Bar\r\nContent-Length: 123\r\n\r\n", response
+    assert_equal "HTTP/1.1 200 OK\r\nFoo: Bar\r\nX-Blah: 123\r\nContent-Length: 0\r\n\r\n", response
   end
 
   def test_send_response_with_body
@@ -43,6 +43,15 @@ class SendResponseTest < MiniTest::Test
     assert_equal "HTTP/1.1 200 OK\r\nContent-Length: 6\r\n\r\nfoobar", response
   end
 
+  def test_send_response_with_frozen_headers_hash
+    i, o = IO.pipe
+    h = {Foo: 'bar'}.freeze
+    H1P.send_response(o, h, 'foo')
+    o.close
+    response = i.read
+    assert_equal "HTTP/1.1 200 OK\r\nFoo: bar\r\nContent-Length: 3\r\n\r\nfoo", response
+  end
+
   def test_send_response_with_big_body
     i, o = IO.pipe
     body = "abcdefg" * 10000
@@ -111,4 +120,21 @@ class SendChunkedResponseTest < MiniTest::Test
     assert_equal "HTTP/1.1 200 OK\r\nFoo: bar\r\nTransfer-Encoding: chunked\r\n\r\n3\r\nfoo\r\n3\r\nbar\r\n3\r\nbaz\r\n0\r\n\r\n", response
     assert_equal len, response.bytesize
   end
+
+  def test_send_chunked_response_with_frozen_headers_hash
+    isrc, osrc = IO.pipe
+    osrc << 'foobarbaz'
+    osrc.close
+
+    i, o = IO.pipe
+    h = { 'Foo' => 'bar' }.freeze
+    len = H1P.send_chunked_response(o, h) do
+      isrc.read(3)
+    end
+    o.close
+
+    response = i.read
+    assert_equal "HTTP/1.1 200 OK\r\nFoo: bar\r\nTransfer-Encoding: chunked\r\n\r\n3\r\nfoo\r\n3\r\nbar\r\n3\r\nbaz\r\n0\r\n\r\n", response
+    assert_equal len, response.bytesize
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: h1p
 version: !ruby/object:Gem::Version
-  version: '0.6'
+  version: 0.6.1
 platform: ruby
 authors:
 - Sharon Rosner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-01-05 00:00:00.000000000 Z
+date: 2023-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler