kcar 0.1.1 → 0.1.2

data/GIT-VERSION-GEN

@@ -1,7 +1,7 @@
 #!/bin/sh
 
 GVF=GIT-VERSION-FILE
-DEF_VER=v0.1.1.GIT
+DEF_VER=v0.1.2.GIT
 
 LF='
 '

data/GNUmakefile

@@ -5,6 +5,7 @@ RAKE = rake
 RAGEL = ragel
 GIT_URL = git://git.bogomips.org/kcar.git
 RLFLAGS = -G2
+RSYNC = rsync
 
 GIT-VERSION-FILE: .FORCE-GIT-VERSION-FILE
 	@./GIT-VERSION-GEN
@@ -183,7 +184,7 @@ publish_doc:
 		TZ=UTC xargs touch -d '1970-01-01 00:00:00' doc/rdoc.css
 	$(MAKE) doc_gz
 	chmod 644 $$(find doc -type f)
-	rsync -av doc/ dcvr:/srv/bogomips/kcar/
+	$(RSYNC) -av doc/ dcvr:/srv/bogomips/kcar/
 	git ls-files | xargs touch
 
 # Create gzip variants of the same timestamp as the original so nginx

data/ext/kcar/kcar.rl

@@ -42,6 +42,7 @@ DEF_MAX_LENGTH(REASON, 256);
 #define UH_FL_INTRAILER 0x10
 #define UH_FL_INCHUNK  0x20
 #define UH_FL_KEEPALIVE 0x40
+#define UH_FL_HASHEADER 0x80
 
 struct http_parser {
   int cs; /* Ragel internal state */
@@ -178,6 +179,8 @@ static void write_value(VALUE hdr, struct http_parser *hp,
   const char *vptr;
   long vlen;
 
+  HP_FL_SET(hp, HASHEADER);
+
   /* Rack does not like Status headers, so we never send them */
   if (CSTR_CASE_EQ(fptr, flen, "status")) {
     hp->cont = Qnil;
@@ -534,6 +537,9 @@ static VALUE body_eof(VALUE self)
 {
   struct http_parser *hp = data_get(self);
 
+  if (!HP_FL_TEST(hp, HASHEADER) && HP_FL_ALL(hp, KEEPALIVE))
+    return Qtrue;
+
   if (HP_FL_TEST(hp, CHUNKED))
     return chunked_eof(hp) ? Qtrue : Qfalse;
 
@@ -560,7 +566,7 @@ static VALUE keepalive(VALUE self)
   struct http_parser *hp = data_get(self);
 
   if (HP_FL_ALL(hp, KEEPALIVE)) {
-    if ( HP_FL_TEST(hp, HASBODY) ) {
+    if (HP_FL_TEST(hp, HASHEADER) && HP_FL_TEST(hp, HASBODY) ) {
       if (HP_FL_TEST(hp, CHUNKED) || (hp->len.content >= 0))
         return Qtrue;
 

data/lib/kcar.rb

@@ -1,8 +1,8 @@
 # -*- encoding: binary -*-
 module Kcar
 
-  # current version, currently 0.1.1
-  VERSION = '0.1.1'
+  # current version, currently 0.1.2
+  VERSION = '0.1.2'
 
   autoload :Response, 'kcar/response'
 end

data/lib/kcar/response.rb

@@ -62,7 +62,9 @@ class Response < Struct.new(:sock, :hdr, :unchunk, :buf, :parser)
   # body.  It may only be called once (usually by a Rack server) as it streams
   # the response body off the our socket object.
   def each(&block)
-    return if parser.body_eof?
+    if parser.body_eof?
+      return
+    end
     if unchunk
       parser.chunked? ? each_unchunk(&block) : each_identity(&block)
     else

data/test/test_parser.rb

@@ -32,7 +32,7 @@ class TestParser < Test::Unit::TestCase
     response = @hp.headers(hdr, buf)
     assert_equal(["200 OK", hdr], response)
     assert hdr.empty?
-    assert ! @hp.keepalive? # no content-length
+    assert @hp.keepalive? # no content-length
   end
 
   def test_parser_status_with_content_length

data/test/test_response.rb

@@ -4,12 +4,44 @@ require 'pp'
 require 'socket'
 require 'kcar'
 require 'digest/sha1'
+$stderr.sync = true
 
 class TestSession < Test::Unit::TestCase
   def setup
     @s, @c = UNIXSocket.pair
   end
 
+  def test_http_status_only_pipelined
+    resp = "HTTP/1.1 404 Not Found\r\n\r\n" \
+           "HTTP/1.1 404 Not Found\r\n\r\n"
+    pid = fork do
+      @s << resp
+      @s.close
+    end
+
+    @s.close
+    @response = Kcar::Response.new(@c)
+    status, headers, body = @response.rack
+    assert_equal status, "404 Not Found"
+    assert_equal({}, headers)
+    tmp = []
+    assert_nothing_raised { body.each { |chunk| tmp << chunk.dup } }
+    assert_equal [], tmp
+    assert @response.parser.keepalive?
+    body.close
+
+    status, headers, body = @response.rack
+    assert_equal status, "404 Not Found"
+    assert_equal({},headers)
+    tmp = []
+    assert_nothing_raised { body.each { |chunk| tmp << chunk.dup } }
+    assert_equal [], tmp
+
+    _, status = Process.waitpid2(pid)
+    assert status.success?
+    body.close
+  end
+
   def test_http_small_pipelined_identity
     resp = "HTTP/1.1 200 OK\r\nContent-Length: 12\r\n\r\nhello world\n" \
            "HTTP/1.1 200 OK\r\nContent-Length: 14\r\n\r\ngoodbye world\n"

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: kcar
 version: !ruby/object:Gem::Version 
-  hash: 25
+  hash: 31
   prerelease: false
   segments: 
   - 0
   - 1
-  - 1
-  version: 0.1.1
+  - 2
+  version: 0.1.2
 platform: ruby
 authors: 
 - kcar hackers
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-07-25 00:00:00 +00:00
+date: 2010-11-30 00:00:00 +00:00
 default_executable: 
 dependencies: []