kcar 0.3.0 → 0.3.1

data/GIT-VERSION-GEN

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

data/lib/kcar/response.rb

@@ -63,21 +63,14 @@ class Kcar::Response
   # 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
-    if @parser.body_eof?
-      return
-    end
+    return if @parser.body_eof?
     if @unchunk
       @parser.chunked? ? each_unchunk { |x| yield x } :
                          each_identity { |x| yield x }
     else
-      if @parser.keepalive?
-        @parser.chunked? ? each_rechunk { |x| yield x } :
-                           each_identity { |x| yield x }
-      else
-        each_until_eof { |x| yield x } # fastest path
-      end
+      @parser.chunked? ? each_rechunk { |x| yield x } :
+                         each_identity { |x| yield x }
     end
-    rescue EOFError
   end
 
   # :stopdoc:
@@ -123,6 +116,7 @@ class Kcar::Response
     begin
       yield dst
     end while @sock.readpartial(READ_SIZE, dst)
+  rescue EOFError
   end
 
   def each_identity

data/test/test_response.rb

@@ -485,7 +485,7 @@ class TestSession < Test::Unit::TestCase
     assert ! @c.closed?
   end
 
-  def test_rack_preserve_chunk_ary_no_keepalive
+  def test_rack_preserve_chunk_no_keepalive
     pid = fork do
       @s << "HTTP/1.1 200 OK\r\n"
       @s << "Connection: close\r\n"
@@ -501,11 +501,36 @@ class TestSession < Test::Unit::TestCase
     tmp = []
     assert ! body.parser.keepalive?
     assert_nothing_raised { body.each { |chunk| tmp << chunk.dup } }
-    assert_equal ["5\r\nhello\r\n0\r\nFoo: bar\r\n\r\n"], tmp
+    assert_equal "5\r\nhello\r\n0\r\nFoo: bar\r\n\r\n", tmp.join
     _, status = Process.waitpid2(pid)
     assert status.success?
     body.close
     assert @c.closed?
   end
 
+  def test_rack_preserve_chunk_no_keepalive
+    s = "HTTP/1.1 200 OK\r\n"
+    s << "Connection: close\r\n"
+    s << "Content-Length: 666\r\n"
+    s << "\r\n"
+    s << "hello"
+    @s.write(s)
+    @response = Kcar::Response.new(@c, [])
+    status, headers, body = @response.rack
+    assert_kind_of Array, headers
+    assert_equal status, "200 OK"
+    tmp = []
+    assert ! body.parser.keepalive?
+
+    closer = Thread.new do
+      Thread.pass until tmp[0]
+      @s.close
+    end
+    assert_raises(EOFError) {
+      body.each { |chunk| tmp << chunk.dup }
+    }
+    assert_nil @c.close
+    assert_equal "hello", tmp[0]
+    assert_nil closer.value
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: kcar
 version: !ruby/object:Gem::Version 
-  hash: 19
+  hash: 17
   prerelease: 
   segments: 
   - 0
   - 3
-  - 0
-  version: 0.3.0
+  - 1
+  version: 0.3.1
 platform: ruby
 authors: 
 - kcar hackers
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-06-07 00:00:00 Z
+date: 2012-09-26 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: wrongdoc
@@ -114,7 +114,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: rainbows
-rubygems_version: 1.8.2
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: bytestream to Rack response converter