picohttp 0.1.0 → 0.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: bba60b163b964308fafcdb14504076d28df6831bb2ebf80c9c3b8676f29f4734
-  data.tar.gz: 43ac8f3c3273fb49360d55d765c5c3ade646b3c503bb18eb23c211f7e4c4109a
+  metadata.gz: 8a11d937822d71f87cd78734c038f1a73a25af902b142958208af81b3397f3be
+  data.tar.gz: 21d3337883257c321b37c602d52c7ab68896343d856bfafaec329476b43c35fe
 SHA512:
-  metadata.gz: 65d914a80ec411ddc53b7e0982f6896f8f1315cd07a8daa01a50cc8d8183fefed315ee51525cef2131a28de00809ed5a1d719cebb5df0785bf03bc218bdc945d
-  data.tar.gz: 9229f345ba7d0945066dd60b81b3fbfbeb3c04e8e4ab1f7b7edc13636654d9f6be562ead8e1a5b3ac479f744342395e6d65f3e5a833522f7787e8fc161318df4
+  metadata.gz: 5e73eead0ddf6378830a445562e827a77f235e1f26d37d18c7956ec790652c9f8efb23b82385d90c512dc47a864b796b8f9f7c2826e5f724dcaae63d45c39d43
+  data.tar.gz: 92893c74ac27005fc38891c49ce4c2a5c72e0f7fd453fce38871ab5b626db8cc10f000263c01573b7cceac331dbf4f5089ee413ec71f3839cede7fa0bc29693e

data/ext/picohttp/extconf.rb

@@ -10,4 +10,7 @@ append_cflags("-fvisibility=hidden")
 # Check for Ractor support
 have_func("rb_ext_ractor_safe", "ruby.h")
 
+# Check for rb_hash_new_capa (Ruby 3.2+)
+have_func("rb_hash_new_capa", "ruby.h")
+
 create_makefile("picohttp/picohttp")

data/ext/picohttp/picohttp.c

@@ -2,6 +2,7 @@
 #include "picohttpparser.h"
 
 #define MAX_HEADER_NAME_LEN 256
+#define ENV_HASH_INITIAL_CAPACITY 64
 
 VALUE rb_mPicohttp;
 VALUE rb_ePicohttpParseError;
@@ -12,6 +13,20 @@ static VALUE rb_str_server_protocol;
 static VALUE rb_str_path_info;
 static VALUE rb_str_query_string;
 static VALUE rb_str_empty;
+static VALUE rb_str_http_1_0;
+static VALUE rb_str_http_1_1;
+
+static VALUE
+http_version_string(int minor_version)
+{
+    if (minor_version == 0) {
+        return rb_str_http_1_0;
+    } else if (minor_version == 1) {
+        return rb_str_http_1_1;
+    } else {
+        return rb_sprintf("HTTP/1.%d", minor_version);
+    }
+}
 
 static VALUE
 header_name_to_env_key(const char *name, size_t name_len)
@@ -20,6 +35,14 @@ header_name_to_env_key(const char *name, size_t name_len)
         rb_raise(rb_ePicohttpParseError, "Header name too long");
     }
 
+    // Special cases for Content-Type and Content-Length (no HTTP_ prefix)
+    if (name_len == 12 && strncasecmp(name, "content-type", 12) == 0) {
+        return rb_interned_str_cstr("CONTENT_TYPE");
+    }
+    if (name_len == 14 && strncasecmp(name, "content-length", 14) == 0) {
+        return rb_interned_str_cstr("CONTENT_LENGTH");
+    }
+
     char env_name[MAX_HEADER_NAME_LEN + 6]; // "HTTP_" + name + null terminator
     strcpy(env_name, "HTTP_");
 
@@ -102,11 +125,15 @@ picohttp_parse_request_env(VALUE self, VALUE str)
         rb_raise(rb_ePicohttpParseError, "Invalid HTTP request");
     }
 
+#ifdef HAVE_RB_HASH_NEW_CAPA
+    VALUE env = rb_hash_new_capa(ENV_HASH_INITIAL_CAPACITY);
+#else
     VALUE env = rb_hash_new();
+#endif
 
     // Standard CGI/Rack environment variables
     rb_hash_aset(env, rb_str_request_method, rb_str_new(method, method_len));
-    rb_hash_aset(env, rb_str_server_protocol, rb_sprintf("HTTP/1.%d", minor_version));
+    rb_hash_aset(env, rb_str_server_protocol, http_version_string(minor_version));
 
     // Parse path and query string in C
     const char *query_start = memchr(path, '?', path_len);
@@ -152,6 +179,8 @@ Init_picohttp(void)
     rb_str_path_info = rb_interned_str_cstr("PATH_INFO");
     rb_str_query_string = rb_interned_str_cstr("QUERY_STRING");
     rb_str_empty = rb_interned_str_cstr("");
+    rb_str_http_1_0 = rb_interned_str_cstr("HTTP/1.0");
+    rb_str_http_1_1 = rb_interned_str_cstr("HTTP/1.1");
 
     // Prevent garbage collection of constants
     rb_gc_register_address(&rb_str_request_method);
@@ -159,4 +188,6 @@ Init_picohttp(void)
     rb_gc_register_address(&rb_str_path_info);
     rb_gc_register_address(&rb_str_query_string);
     rb_gc_register_address(&rb_str_empty);
+    rb_gc_register_address(&rb_str_http_1_0);
+    rb_gc_register_address(&rb_str_http_1_1);
 }

data/lib/picohttp/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Picohttp
-  VERSION = "0.1.0"
+  VERSION = "0.1.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: picohttp
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
 platform: ruby
 authors:
 - John Hawthorn