h1p 0.6.1 → 1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 96197142deeb30d27fe9b3fc9de21a07ed1de8f0a1d3db9b8d967310056bdd12
-  data.tar.gz: aa71893e95f7a7e8ba8de8123987445262edd6dbcad7efba12337091a2421d14
+  metadata.gz: 2d2493b6fc11d7deb1231f9a2d0bf1c6dc59b00a01065da7086e8c65326f74df
+  data.tar.gz: e30bf2c5a3213a4744a66a412e2db34a2b26b05baa8073409a18c7e0bd314de3
 SHA512:
-  metadata.gz: 9380e218a2433f61a88676fb807ecb742391dff2c6d7e4937d24ff1385839ed897eaf81eb3d8d1f916bcf53f30cd2fd9c31f0c78fcd6c0e35d3753dc1d3a1252
-  data.tar.gz: 2ce69a5e542390018f779e9ba0a73263b362b3a6d69838494da73d18cb6f84a3262ed98fd086bb098ffa253f264db4d3e1d40e5ae8f9757ca1928c36ce9f02ce
+  metadata.gz: 2adc85f461d3a41e7e67f9c199be486b85fd46407e60455329ac065a5779c60d62328a049091bb9d4b7d0be2f1610db38a8744c35f7071b06991ec837235a981
+  data.tar.gz: 926acbd38590f74666e8d774b6f54d3b610cbf0570495727049b2c2ba628a80888d30e2e0bd83ac26b3897c9a0875be42ce336a439a9e07a2dfb8160aebc8441

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.0 2023-06-07
+
+- Add support for array as header value (#2)
+
 ## 0.6.1 2023-05-28
 
 - Fix sending response with frozen headers hash

data/Gemfile.lock

@@ -1,14 +1,14 @@
 PATH
   remote: .
   specs:
-    h1p (0.6.1)
+    h1p (1.0)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    minitest (5.17.0)
+    minitest (5.18.0)
     rake (13.0.6)
-    rake-compiler (1.2.1)
+    rake-compiler (1.2.3)
       rake
 
 PLATFORMS
@@ -16,9 +16,9 @@ PLATFORMS
 
 DEPENDENCIES
   h1p!
-  minitest (~> 5.17.0)
+  minitest (~> 5.18.0)
   rake (~> 13.0.6)
-  rake-compiler (= 1.2.1)
+  rake-compiler (= 1.2.3)
 
 BUNDLED WITH
    2.2.26

data/ext/h1p/h1p.c

@@ -22,6 +22,7 @@ ID ID_call;
 ID ID_downcase;
 ID ID_eof_p;
 ID ID_eq;
+ID ID_join;
 ID ID_read_method;
 ID ID_read;
 ID ID_readpartial;
@@ -56,6 +57,7 @@ VALUE STR_transfer_encoding_capitalized;
 
 VALUE STR_CRLF;
 VALUE STR_EMPTY_CHUNK;
+VALUE STR_COMMA_SPACE;
 
 VALUE SYM_backend_read;
 VALUE SYM_backend_recv;
@@ -1176,12 +1178,24 @@ void send_response_write_status_line(send_response_ctx *ctx, VALUE protocol, VAL
   ctx->buffer_len += partlen + 2;
 }
 
+inline static VALUE format_comma_separated_header_values(VALUE array) {
+  return rb_funcall(array, ID_join, 1, STR_COMMA_SPACE);
+}
+
 int send_response_write_header(VALUE key, VALUE val, VALUE arg) {
   if (TYPE(key) != T_STRING) key = rb_funcall(key, ID_to_s, 0);
   char *keyptr = RSTRING_PTR(key);
   if (RSTRING_LEN(key) < 1 || keyptr[0] == ':') return 0;
 
-  if (TYPE(val) != T_STRING) val = rb_funcall(val, ID_to_s, 0);
+  switch (TYPE(val)) {
+    case T_STRING:
+      break;
+    case T_ARRAY:
+      val = format_comma_separated_header_values(val);
+      break;
+    default:
+      val = rb_funcall(val, ID_to_s, 0);
+  }
   unsigned int keylen = RSTRING_LEN(key);
   char *valptr = RSTRING_PTR(val);
   unsigned int vallen = RSTRING_LEN(val);
@@ -1377,6 +1391,7 @@ void Init_H1P(void) {
   ID_downcase               = rb_intern("downcase");
   ID_eof_p                  = rb_intern("eof?");
   ID_eq                     = rb_intern("==");
+  ID_join                   = rb_intern("join");
   ID_read_method            = rb_intern("__read_method__");
   ID_read                   = rb_intern("read");
   ID_readpartial            = rb_intern("readpartial");
@@ -1405,8 +1420,9 @@ void Init_H1P(void) {
   GLOBAL_STR(STR_transfer_encoding,             "transfer-encoding");
   GLOBAL_STR(STR_transfer_encoding_capitalized, "Transfer-Encoding");
 
-  GLOBAL_STR(STR_CRLF,                        "\r\n");
-  GLOBAL_STR(STR_EMPTY_CHUNK,                 "0\r\n\r\n");
+  GLOBAL_STR(STR_CRLF,                          "\r\n");
+  GLOBAL_STR(STR_EMPTY_CHUNK,                   "0\r\n\r\n");
+  GLOBAL_STR(STR_COMMA_SPACE,                   ", ");
 
   SYM_backend_read  = ID2SYM(ID_backend_read);
   SYM_backend_recv  = ID2SYM(ID_backend_recv);

data/h1p.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |s|
   s.require_paths = ["lib"]
   s.required_ruby_version = '>= 2.7'
 
-  s.add_development_dependency  'rake-compiler',        '1.2.1'
+  s.add_development_dependency  'rake-compiler',        '1.2.3'
   s.add_development_dependency  'rake',               '~>13.0.6'
-  s.add_development_dependency  'minitest',           '~>5.17.0'
+  s.add_development_dependency  'minitest',           '~>5.18.0'
 end

data/lib/h1p/version.rb

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

data/test/test_h1p.rb

@@ -35,6 +35,14 @@ class SendResponseTest < MiniTest::Test
     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_multiple_header_values
+    i, o = IO.pipe
+    H1P.send_response(o, { :Foo => ['Bar', 'Baz'], 'X-Blah' => 123 })
+    o.close
+    response = i.read
+    assert_equal "HTTP/1.1 200 OK\r\nFoo: Bar, Baz\r\nX-Blah: 123\r\nContent-Length: 0\r\n\r\n", response
+  end
+
   def test_send_response_with_body
     i, o = IO.pipe
     H1P.send_response(o, {}, "foobar")

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: h1p
 version: !ruby/object:Gem::Version
-  version: 0.6.1
+  version: '1.0'
 platform: ruby
 authors:
 - Sharon Rosner
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-05-28 00:00:00.000000000 Z
+date: 2023-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake-compiler
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: 1.2.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: 1.2.3
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -44,14 +44,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.17.0
+        version: 5.18.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.17.0
+        version: 5.18.0
 description: 
 email: sharon@noteflakes.com
 executables: []