yahns 1.12.5 → 1.13.0

data/man/yahns_config.5

@@ -1,4 +1,4 @@
-.\" Automatically generated by Pod::Man 2.25 (Pod::Simple 3.16)
+.\" Automatically generated by Pod::Man 2.28 (Pod::Simple 3.28)
 .\"
 .\" Standard preamble:
 .\" ========================================================================
@@ -38,6 +38,8 @@
 .    ds PI \(*p
 .    ds L" ``
 .    ds R" ''
+.    ds C`
+.    ds C'
 'br\}
 .\"
 .\" Escape single quotes in literal strings from groff's Unicode transform.
@@ -48,17 +50,24 @@
 .\" titles (.TH), headers (.SH), subsections (.SS), items (.Ip), and index
 .\" entries marked with X<> in POD.  Of course, you'll have to process the
 .\" output yourself in some meaningful fashion.
-.ie \nF \{\
-.    de IX
-.    tm Index:\\$1\t\\n%\t"\\$2"
+.\"
+.\" Avoid warning from groff about undefined register 'F'.
+.de IX
 ..
-.    nr % 0
-.    rr F
-.\}
-.el \{\
-.    de IX
+.nr rF 0
+.if \n(.g .if rF .nr rF 1
+.if (\n(rF:(\n(.g==0)) \{
+.    if \nF \{
+.        de IX
+.        tm Index:\\$1\t\\n%\t"\\$2"
 ..
+.        if !\nF==2 \{
+.            nr % 0
+.            nr F 2
+.        \}
+.    \}
 .\}
+.rr rF
 .\"
 .\" Accent mark definitions (@(#)ms.acc 1.5 88/02/08 SMI; from UCB 4.2).
 .\" Fear.  Run.  Save yourself.  No user-serviceable parts.
@@ -124,7 +133,7 @@
 .\" ========================================================================
 .\"
 .IX Title "YAHNS_CONFIG 5"
-.TH YAHNS_CONFIG 5 "1994-10-02" "yahns 1.12.4.8.gd5d0" "yahns user manual"
+.TH YAHNS_CONFIG 5 "1994-10-02" "yahns 1.12.5.48.g013d" "yahns user manual"
 .\" For nroff, turn off justification.  Always turn off hyphenation; it makes
 .\" way too many mistakes in technical documents.
 .if n .ad l
@@ -152,7 +161,7 @@ An app with the same :TYPE and \s-1APP_ARGUMENTS\s0 may be defined multiple
 times with different &BLOCK contents (with different listeners and
 buffering settings).
 .Sp
-See the \*(L"APP-LEVEL \s-1DIRECTIVES\s0\*(R" section for details on what goes into
+See the \*(L"APP-LEVEL \s-1DIRECTIVES\*(R"\s0 section for details on what goes into
 the &BLOCK.
 .Sp
 This directive may be specified multiple times for different or
@@ -161,7 +170,7 @@ identical applications.
 If the \*(L"working_directory\*(R" directive is used, all app directives may
 only be specified after setting \*(L"working_directory\*(R".
 .Sp
-For Rack \s-1HTTP\s0 applications, see \*(L"\s-1RACK\s0 \s-1APP\s0 \s-1ARGUMENTS\s0\*(R" for more
+For Rack \s-1HTTP\s0 applications, see \*(L"\s-1RACK APP ARGUMENTS\*(R"\s0 for more
 information.
 .IP "before_exec &BLOCK" 4
 .IX Item "before_exec &BLOCK"
@@ -227,7 +236,7 @@ Default: none
 .IX Item "queue [NAME] &BLOCK"
 As a top-level directive, this configures or defines a queue.
 If no \s-1NAME\s0 is specified, a default queue (named :default) is
-assumed.  See the \*(L"QUEUE-LEVEL \s-1DIRECTIVES\s0\*(R" section for details.
+assumed.  See the \*(L"QUEUE-LEVEL \s-1DIRECTIVES\*(R"\s0 section for details.
 .Sp
 A &BLOCK must be given if used as a top-level directive.  This
 behaves slightly differently inside an app &BLOCK.  This may also
@@ -297,8 +306,8 @@ Default: 7
 .IX Subsection "APP-LEVEL DIRECTIVES"
 .IP "atfork_prepare, atfork_parent, atfork_child" 4
 .IX Item "atfork_prepare, atfork_parent, atfork_child"
-These are identical to the methods defined in \s-1WORKER_PROCESSES\-LEVEL\s0
-\&\s-1DIRECTIVES\s0, however they are available inside the app blocks for
+These are identical to the methods defined in \s-1WORKER_PROCESSES\-LEVEL
+DIRECTIVES,\s0 however they are available inside the app blocks for
 convenience in case it is easier to organize per-app hooks.
 .Sp
 Default: (none)
@@ -315,7 +324,7 @@ This only affects clients connecting over Unix domain sockets and
 \&\s-1TCP\s0 via loopback (127.*.*.*).  It is unlikely to detect disconnects
 if the client is on a remote host (even on a fast \s-1LAN\s0).
 .Sp
-This has no effect for (rare) \s-1HTTP/0\s0.9 clients.
+This has no effect for (rare) \s-1HTTP/0.9\s0 clients.
 .Sp
 Default: false
 .IP "client_body_buffer_size \s-1INTEGER\s0" 4
@@ -340,7 +349,7 @@ Default: 4000 bytes
 .IP "client_max_body_size {INTEGER|nil}" 4
 .IX Item "client_max_body_size {INTEGER|nil}"
 This controls the maximum request body size before a client is
-rejected with an \s-1HTTP\s0 413 error.
+rejected with an \s-1HTTP 413\s0 error.
 .Sp
 Setting this to nil will allow unlimited-sized inputs.
 .Sp
@@ -367,7 +376,7 @@ Default: 15 (seconds)
 .IP "errors {IO|PATHNAME}" 4
 .IX Item "errors {IO|PATHNAME}"
 For Rack applications, this controls the env[\*(L"rack.errors\*(R"]
-destination.  If given a \s-1PATHNAME\s0, it will be a writable file
+destination.  If given a \s-1PATHNAME,\s0 it will be a writable file
 opened with O_APPEND without userspace buffering, making it suitable
 for concurrent appends by multiple processes and threads, as well as
 making it eligible for reopening via \s-1SIGUSR1\s0 after log rotation.
@@ -421,7 +430,7 @@ Default: Dir.tmpdir (usually from \s-1TMPDIR\s0 env or /tmp)
 .IX Item "listen ADDRESS [, OPTIONS]"
 Adds an \s-1ADDRESS\s0 to the existing listener set.  May be specified more
 than once.  \s-1ADDRESS\s0 may be an Integer port number for a \s-1TCP\s0 port, an
-\&\*(L"\s-1IP_ADDRESS:PORT\s0\*(R" for \s-1TCP\s0 listeners or a pathname for \s-1UNIX\s0 domain sockets.
+\&\*(L"\s-1IP_ADDRESS:PORT\*(R"\s0 for \s-1TCP\s0 listeners or a pathname for \s-1UNIX\s0 domain sockets.
 .Sp
 .Vb 2
 \&  # listen to port 3000 on all TCP interfaces
@@ -476,7 +485,7 @@ listener is required if this is true.
 .Sp
 Enabling this option for the IPv6\-only listener and having a
 separate IPv4 listener is recommended if you wish to support IPv6
-on the same \s-1TCP\s0 port.  Otherwise, the value of env[\*(L"\s-1REMOTE_ADDR\s0\*(R"]
+on the same \s-1TCP\s0 port.  Otherwise, the value of env[\*(L"\s-1REMOTE_ADDR\*(R"\s0]
 will appear as an ugly IPv4\-mapped\-IPv6 address for IPv4 clients
 (e.g \*(L":ffff:10.0.0.1\*(R" instead of just \*(L"10.0.0.1\*(R").
 .Sp
@@ -502,9 +511,9 @@ This enables multiple, independently-started yahns instances to
 bind to the same port (as long as all the processes enable this).
 .Sp
 This option must be used when yahns first binds the listen socket.
-It cannot be enabled when a socket is inherited via \s-1SIGUSR2\s0
-(but it will remain on if inherited), and it cannot be enabled
-directly via \s-1SIGHUP\s0.
+It cannot be enabled when a socket is inherited via \s-1SIGUSR2
+\&\s0(but it will remain on if inherited), and it cannot be enabled
+directly via \s-1SIGHUP.\s0
 .Sp
 Note: there is a chance of connections being dropped if
 one of the yahns instances is stopped while using this.
@@ -520,7 +529,7 @@ To enable \s-1TLS\s0 connections, you must configure this yourself.
 See documentation for OpenSSL::SSL::SSLContext
 for more information:
 .Sp
-http://docs.ruby\-lang.org/en/trunk/OpenSSL/SSL/SSLContext.html <http://docs.ruby-lang.org/en/trunk/OpenSSL/SSL/SSLContext.html>
+<http://docs.ruby\-lang.org/en/trunk/OpenSSL/SSL/SSLContext.html>
 .Sp
 Default: none
 .Sp
@@ -545,6 +554,9 @@ An example which seems to work is:
 \&  # but disable client certificate verification as it is rare:
 \&  ssl_ctx.set_params(verify_mode: OpenSSL::SSL::VERIFY_NONE)
 \&
+\&  # Built\-in session cache (only works if worker_processes is nil or 1)
+\&  ssl_ctx.session_cache_mode = OpenSSL::SSL::SSLContext::SESSION_CACHE_SERVER
+\&
 \&  app(:rack, "/path/to/my/app/config.ru") do
 \&    listen 443, ssl_ctx: ssl_ctx
 \&  end
@@ -624,7 +636,7 @@ client_expire_threshold is hit and a client hits its client_timeout.
 Default: true
 .IP "user \s-1USER\s0 [,GROUP]" 4
 .IX Item "user USER [,GROUP]"
-Runs application process(es) as the specified \s-1USER\s0 and \s-1GROUP\s0.
+Runs application process(es) as the specified \s-1USER\s0 and \s-1GROUP.\s0
 .Sp
 If using worker_processes, this only affects the workers and the
 master stays running as the user who started it.  This switch will
@@ -641,7 +653,7 @@ existing queue.
 If given a &BLOCK\-only, this will create an anonymous queue for this
 application context only.  If given a &BLOCK, this takes the same
 parameters (worker_threads and max_events) as described in
-\&\*(L"QUEUE-LEVEL \s-1DIRECTIVES\s0\*(R".
+\&\*(L"QUEUE-LEVEL \s-1DIRECTIVES\*(R"\s0.
 .Sp
 \&\s-1NAME\s0 and &BLOCK may not be combined inside an app &BLOCK.
 .Sp
@@ -665,16 +677,16 @@ model instead of a single process.
 .Sp
 If an optional &BLOCK is given, it may be used to configure
 \&\fIpthread_atfork\fR\|(3)\-style hooks.
-See \*(L"\s-1WORKER_PROCESSES\-LEVEL\s0 \s-1DIRECTIVES\s0\*(R" for details.
+See \*(L"\s-1WORKER_PROCESSES\-LEVEL DIRECTIVES\*(R"\s0 for details.
 .Sp
 Using worker_processes is strongly recommended if your application
 relies on using a \s-1SIGCHLD\s0 handler for reaping forked processes.
 Without worker_processes, yahns must reserve \s-1SIGCHLD\s0 for rolling
 back \s-1SIGUSR2\s0 upgrades, leading to conflicts if the appplication
-expects to handle \s-1SIGCHLD\s0.
+expects to handle \s-1SIGCHLD.\s0
 .Sp
 Default: nil (single process, no master/worker separation)
-.SS "\s-1WORKER_PROCESSES\-LEVEL\s0 \s-1DIRECTIVES\s0"
+.SS "\s-1WORKER_PROCESSES\-LEVEL DIRECTIVES\s0"
 .IX Subsection "WORKER_PROCESSES-LEVEL DIRECTIVES"
 Note: all of the atfork_* hooks described here are available inside the
 \&\*(L"app\*(R" blocks, too.
@@ -709,11 +721,11 @@ The only supported keyword argument is:
 .IP "preload: \s-1BOOLEAN\s0" 4
 .IX Item "preload: BOOLEAN"
 preload: only makes sense if worker_processes are configured.
-Preloading an app allows memory to be saved under a copy-on-write \s-1GC\s0,
+Preloading an app allows memory to be saved under a copy-on-write \s-1GC,\s0
 but will often require atfork_* hooks to be registered when configuring
 worker_processes.  preload: defaults to false for maximum out-of-the-box
 compatibility.
-.SS "\s-1RACK\s0 \s-1APP\s0 \s-1EXAMPLE\s0"
+.SS "\s-1RACK APP EXAMPLE\s0"
 .IX Subsection "RACK APP EXAMPLE"
 .Vb 3
 \&  app(:rack, "/path/to/config.ru", preload: false) do
@@ -729,13 +741,13 @@ See the examples/ directory in the git source tree.
 .Ve
 .SH "CONTACT"
 .IX Header "CONTACT"
-All feedback welcome via plain-text mail to mailto:yahns\-public@yhbt.net <mailto:yahns-public@yhbt.net>
+All feedback welcome via plain-text mail to <mailto:yahns\-public@yhbt.net>
 No subscription is necessary to post to the mailing list.
-List archives are available at http://yhbt.net/yahns\-public/ <http://yhbt.net/yahns-public/>
+List archives are available at <http://yhbt.net/yahns\-public/>
 .SH "COPYRIGHT"
 .IX Header "COPYRIGHT"
-Copyright (C) 2013\-2016 all contributors mailto:yahns\-public@yhbt.net <mailto:yahns-public@yhbt.net>
-License: \s-1GPL\-3\s0.0+ http://www.gnu.org/licenses/gpl\-3.0.txt <http://www.gnu.org/licenses/gpl-3.0.txt>
+Copyright (C) 2013\-2016 all contributors <mailto:yahns\-public@yhbt.net>
+License: \s-1GPL\-3.0+ \s0<http://www.gnu.org/licenses/gpl\-3.0.txt>
 .SH "SEE ALSO"
 .IX Header "SEE ALSO"
 \&\fIyahns\fR\|(1)

data/test/helper.rb

@@ -147,6 +147,18 @@ class DieIfUsed
   end
 end
 
+# tricky to test output buffering behavior across different OSes
+def skip_skb_mem
+  return if ENV['YAHNS_TEST_FORCE']
+  skip "linux-only test" unless RUBY_PLATFORM =~ /linux/
+  [ [ '/proc/sys/net/ipv4/tcp_rmem', "4096	87380	6291456\n" ],
+    [ '/proc/sys/net/ipv4/tcp_wmem', "4096	16384	4194304\n" ]
+  ].each do |file, expect|
+    val = IO.read(file)
+    val == expect or skip "#{file} had: #{val}expected: #{expect}"
+  end
+end
+
 require 'yahns'
 
 # needed for parallel (MT) tests)

data/test/test_auto_chunk.rb

@@ -0,0 +1,56 @@
+# Copyright (C) 2013-2016 all contributors <yahns-public@yhbt.net>
+# License: GPL-3.0+ <https://www.gnu.org/licenses/gpl-3.0.txt>
+# frozen_string_literal: true
+require_relative 'server_helper'
+
+class TestAutoChunk < Testcase
+  ENV["N"].to_i > 1 and parallelize_me!
+  include ServerHelper
+  alias setup server_helper_setup
+  alias teardown server_helper_teardown
+
+  def test_auto_head
+    err = @err
+    cfg = Yahns::Config.new
+    host, port = @srv.addr[3], @srv.addr[1]
+    cfg.instance_eval do
+      GTL.synchronize do
+        app = Rack::Builder.new do
+          use Rack::ContentType, "text/plain"
+          run(lambda do |env|
+            [ 200, {}, %w(a b c) ]
+          end)
+        end
+        app(:rack, app) { listen "#{host}:#{port}" }
+      end
+      logger(Logger.new(err.path))
+    end
+    pid = mkserver(cfg)
+    s = TCPSocket.new(host, port)
+    s.write("GET / HTTP/1.0\r\n\r\n")
+    assert s.wait(30), "IO wait failed"
+    buf = s.read
+    assert_match %r{\r\n\r\nabc\z}, buf
+    s.close
+
+    s = TCPSocket.new(host, port)
+    s.write("GET / HTTP/1.1\r\nHost: example.com\r\n\r\n")
+    buf = ''.dup
+    Timeout.timeout(30) do
+      until buf =~ /\r\n\r\n1\r\na\r\n1\r\nb\r\n1\r\nc\r\n0\r\n\r\n\z/
+        buf << s.readpartial(16384)
+      end
+    end
+    assert_match(%r{^Transfer-Encoding: chunked\r\n}, buf)
+    s.close
+
+    Net::HTTP.start(host, port) do |http|
+      req = Net::HTTP::Get.new("/")
+      res = http.request(req)
+      assert_equal 200, res.code.to_i
+      assert_equal 'abc', res.body
+    end
+  ensure
+    quit_wait(pid)
+  end
+end

data/test/test_extras_exec_cgi.rb

@@ -169,10 +169,8 @@ class TestExtrasExecCGI < Testcase
         assert_match %r{200 OK}, head
         assert_match %r{\A\d+\n\z}, body
         exec_pid = body.to_i
-        assert_raises(Errno::EAGAIN, IO::WaitReadable) { c.read_nonblock(666) }
         poke_until_dead exec_pid
-        # still alive?
-        assert_raises(Errno::EAGAIN, IO::WaitReadable) { c.read_nonblock(666) }
+        assert_raises(EOFError) { c.read_nonblock(666) }
       else
         raise "BUG in test, bad rtype"
       end

data/test/test_extras_try_gzip_static.rb

@@ -34,8 +34,22 @@ class TestExtrasTryGzipStatic < Testcase
       end
     end
 
+    Net::HTTP.start(host, port) do |http|
+      uri = "/COPYING/foo" + ('-' * 4096)
+      begin
+        res = http.request(Net::HTTP::Get.new(uri))
+      end while res.code.to_i == 414 && uri.chop!
+      res = http.request(Net::HTTP::Get.new("/COPYING/foo"))
+      assert_equal 404, res.code.to_i
+      lines = File.readlines(err.path)
+      File.truncate(err.path, 0)
+      assert_operator lines.size, :<, 3, lines.map! { |s| s[0,64] }.inspect
+    end
+
     begin # setup
       gpl = "#{tmpdir}/COPYING"
+      File.symlink gpl, "#{tmpdir}/COPYING.abssymlink"
+      File.symlink "COPYING", "#{tmpdir}/COPYING.relsymlink"
       gplgz = "#{tmpdir}/COPYING.gz"
       FileUtils.cp("COPYING", gpl)
       _, status = Process.waitpid2(fork do
@@ -76,16 +90,19 @@ class TestExtrasTryGzipStatic < Testcase
         when "HEAD" then assert_nil body
         end
 
-        req = "#{m} /COPYING HTTP/1.0\r\nAccept-Encoding: gzip"
-        body = check.call(req) do |head|
-          assert_match %r{^Content-Encoding: gzip\b}, head
-          assert_match %r{^Content-Type: text/plain\b}, head
-          assert_match %r{^Content-Length: #{gz_st.size}\b}, head
-        end
-        case m
-        when "GET"
-          assert_equal GPL_TEXT, Zlib::GzipReader.new(StringIO.new(body)).read
-        when "HEAD" then assert_nil body
+        %w(COPYING COPYING.abssymlink COPYING.relsymlink).each do |path|
+          req = "#{m} /#{path} HTTP/1.0\r\nAccept-Encoding: gzip"
+          body = check.call(req) do |head|
+            assert_match %r{^Content-Encoding: gzip\b}, head
+            assert_match %r{^Content-Type: text/plain\b}, head
+            assert_match %r{^Content-Length: #{gz_st.size}\b}, head
+          end
+          case m
+          when "GET"
+            body =StringIO.new(body)
+            assert_equal GPL_TEXT, Zlib::GzipReader.new(body).read
+          when "HEAD" then assert_nil body
+          end
         end
       end
     end
@@ -108,11 +125,10 @@ class TestExtrasTryGzipStatic < Testcase
 
         req = "#{m} /COPYING HTTP/1.0\r\n" \
               "Range: bytes=66666666-\r\nAccept-Encoding: gzip"
-        body = check.call(req) do |head|
+        check.call(req) do |head|
           assert_match %r{^Content-Range: bytes \*/#{st.size}\r\n}, head
           assert_match %r{\AHTTP/1\.1 416 }, head
         end
-        assert_nil body
       end
     end
 
@@ -164,15 +180,13 @@ class TestExtrasTryGzipStatic < Testcase
     Timeout.timeout(30) do # 404
       %w(GET HEAD).each do |m|
         req = "#{m} /cp-ing HTTP/1.0\r\nAccept-Encoding: gzip"
-        body = check.call(req) do |head|
+        check.call(req) do |head|
           assert_match %r{HTTP/1\.1 404 }, head
         end
-        assert_nil body
       end
-      body = check.call("FOO /COPYING HTTP/1.0") do |head|
+      check.call("FOO /COPYING HTTP/1.0") do |head|
         assert_match %r{HTTP/1\.1 405 }, head
       end
-      assert_nil body
     end
 
     Net::HTTP.start(host, port) do |http|

data/test/test_output_buffering.rb

@@ -126,7 +126,10 @@ class TestOutputBuffering < Testcase
           end
           size = gplv3.size
           len = size.to_s
-          ranges = Rack::Utils.byte_ranges(e, size)
+
+          ranges = Rack::Utils.respond_to?(:get_byte_ranges) ?
+                   Rack::Utils.get_byte_ranges(e['HTTP_RANGE'], size) :
+                   Rack::Utils.byte_ranges(e, size)
           status = 200
           h = { "Content-Type" => "text/plain", "Content-Length" => len }
           if ranges && ranges.size == 1
@@ -216,6 +219,7 @@ class TestOutputBuffering < Testcase
 
   def test_client_timeout
     err = @err
+    skip_skb_mem
     apperr = tmpfile(%w(app .err))
     cfg = Yahns::Config.new
     size = RAND.size * NR

data/test/test_proxy_pass.rb

@@ -485,7 +485,7 @@ class TestProxyPass < Testcase
     assert_equal exp, res
     errs = File.readlines(@err.path).grep(/\bERROR\b/)
     assert_equal 1, errs.size
-    assert_match(/premature upstream EOF/, errs[0])
+    assert_match(/upstream EOF/, errs[0])
     @err.truncate(0)
 
     # truncated headers or no response at all...
@@ -501,7 +501,7 @@ class TestProxyPass < Testcase
       s.close
       errs = File.readlines(@err.path).grep(/\bERROR\b/)
       assert_equal 1, errs.size
-      assert_match(/premature upstream EOF/, errs[0])
+      assert_match(/upstream EOF/, errs[0])
       @err.truncate(0)
     end
   end

data/test/test_proxy_pass_no_buffering.rb

@@ -0,0 +1,170 @@
+# Copyright (C) 2015-2016 all contributors <yahns-public@yhbt.net>
+# License: GPL-3.0+ (https://www.gnu.org/licenses/gpl-3.0.txt)
+# frozen_string_literal: true
+require_relative 'server_helper'
+begin
+  require 'kcar'
+rescue LoadError
+end
+require 'digest/md5'
+class TestProxyPassNoBuffering < Testcase
+  ENV["N"].to_i > 1 and parallelize_me!
+  include ServerHelper
+  STR4 = 'abcd' * (256 * 1024)
+  NCHUNK = 50
+  class ProxiedApp
+    def call(env)
+      case env['REQUEST_METHOD']
+      when 'GET'
+        case env['PATH_INFO']
+        when '/giant-body'
+          h = [ %W(content-type text/pain) ]
+
+          # HTTP/1.0 is not Rack-compliant, so no Rack::Lint for us :)
+          if env['HTTP_VERSION'] == 'HTTP/1.1'
+            h << %W(content-length #{NCHUNK * STR4.size})
+          end
+
+          body = Object.new
+          def body.each
+            NCHUNK.times { yield STR4 }
+          end
+          [ 200, h, body ]
+        end
+      end
+    end
+  end
+
+  def setup
+    @srv2 = TCPServer.new(ENV["TEST_HOST"] || "127.0.0.1", 0)
+    server_helper_setup
+    skip "kcar missing yahns/proxy_pass" unless defined?(Kcar)
+    require 'yahns/proxy_pass'
+    @tmpdir = yahns_mktmpdir
+  end
+
+  def teardown
+    @srv2.close if defined?(@srv2) && !@srv2.closed?
+    server_helper_teardown
+    FileUtils.rm_rf(@tmpdir) if defined?(@tmpdir)
+  end
+
+  def check_headers(io)
+    l = io.gets
+    assert_match %r{\AHTTP/1\.[01] 200\b}, l
+    begin
+      l = io.gets
+    end until l == "\r\n"
+  end
+
+  def test_proxy_pass_no_buffering
+    to_close = []
+    err, cfg, host, port = @err, Yahns::Config.new, @srv.addr[3], @srv.addr[1]
+    host2, port2 = @srv2.addr[3], @srv2.addr[1]
+    pxp = Yahns::ProxyPass.new("http://#{host2}:#{port2}",
+                               proxy_buffering: false)
+    tmpdir = @tmpdir
+    pid = mkserver(cfg) do
+      ObjectSpace.each_object(Yahns::TmpIO) { |io| io.close unless io.closed? }
+      @srv2.close
+      cfg.instance_eval do
+        app(:rack, pxp) do
+          listen "#{host}:#{port}"
+          output_buffering true, tmpdir: tmpdir
+        end
+        stderr_path err.path
+      end
+    end
+
+    pid2 = mkserver(cfg, @srv2) do
+      ObjectSpace.each_object(Yahns::TmpIO) { |io| io.close unless io.closed? }
+      @srv.close
+      cfg.instance_eval do
+        app(:rack, ProxiedApp.new) do
+          output_buffering false
+          listen "#{host2}:#{port2}"
+        end
+        stderr_path err.path
+      end
+    end
+    %w(1.0 1.1).each do |ver|
+      s = TCPSocket.new(host, port)
+      to_close << s
+      req = "GET /giant-body HTTP/#{ver}\r\nHost: example.com\r\n".dup
+      req << "Connection: close\r\n" if ver == '1.1'
+      req << "\r\n"
+      s.write(req)
+      bufs = []
+      sleep 1
+      10.times do
+        sleep 0.1
+        # ensure no files get created
+        if RUBY_PLATFORM =~ /\blinux\b/ && `which lsof 2>/dev/null`.size >= 4
+          qtmpdir = Regexp.quote("#@tmpdir/")
+          deleted1 = `lsof -p #{pid}`.split("\n")
+          deleted1 = deleted1.grep(/\bREG\b.*#{qtmpdir}.* \(deleted\)/)
+          deleted2 = `lsof -p #{pid2}`.split("\n")
+          deleted2 = deleted2.grep(/\bREG\b.*#{qtmpdir}.* \(deleted\)/)
+          [ deleted1, deleted2 ].each do |ary|
+            ary.delete_if { |x| x =~ /\.(?:err|out|rb|ru) \(deleted\)/ }
+          end
+          assert_equal 0, deleted1.size, "pid1=#{deleted1.inspect}"
+          assert_equal 0, deleted2.size, "pid2=#{deleted2.inspect}"
+          bufs.push(deleted1[0])
+        end
+      end
+      before = bufs.size
+      bufs.uniq!
+      assert bufs.size < before, 'unlinked buffer should not grow'
+      buf = ''.dup
+      slow = Digest::MD5.new
+      ft = Thread.new do
+        fast = Digest::MD5.new
+        f = TCPSocket.new(host2, port2)
+        f.write(req)
+        b2 = ''.dup
+        check_headers(f)
+        nf = 0
+        begin
+          f.readpartial(1024 * 1024, b2)
+          nf += b2.bytesize
+          fast.update(b2)
+        rescue EOFError
+          f = f.close
+        end while f
+        b2.clear
+        [ nf, fast.hexdigest ]
+      end
+      Thread.abort_on_exception = true
+      check_headers(s)
+      n = 0
+      begin
+        s.readpartial(1024 * 1024, buf)
+        slow.update(buf)
+        n += buf.bytesize
+        sleep 0.01
+      rescue EOFError
+        s = s.close
+      end while s
+      ft.join(5)
+      assert_equal [n, slow.hexdigest ], ft.value
+
+      fast = Digest::MD5.new
+      f = TCPSocket.new(host, port)
+      f.write(req)
+      check_headers(f)
+      begin
+        f.readpartial(1024 * 1024, buf)
+        fast.update(buf)
+      rescue EOFError
+        f = f.close
+      end while f
+      buf.clear
+      assert_equal slow.hexdigest, fast.hexdigest
+    end
+  ensure
+    to_close.each { |io| io.close unless io.closed? }
+    quit_wait(pid)
+    quit_wait(pid2)
+  end
+end

data/test/test_reopen_logs.rb

@@ -2,7 +2,11 @@
 # License: GPL-3.0+ (https://www.gnu.org/licenses/gpl-3.0.txt)
 # frozen_string_literal: true
 require_relative 'server_helper'
-require 'rack/commonlogger'
+require 'rack'
+# trigger autoload, since rack 1.x => 2.x renames:
+# 'rack/commonlogger' => 'rack/common_logger'
+# so we can't require directly
+Rack::CommonLogger.class
 
 class TestReopenLogs < Testcase
   ENV["N"].to_i > 1 and parallelize_me!

data/test/test_response.rb

@@ -9,6 +9,48 @@ class TestResponse < Testcase
   alias setup server_helper_setup
   alias teardown server_helper_teardown
 
+  def test_auto_head
+    err = @err
+    cfg = Yahns::Config.new
+    host, port = @srv.addr[3], @srv.addr[1]
+    str = "HELLO WORLD\n"
+    cfg.instance_eval do
+      GTL.synchronize do
+        app = Rack::Builder.new do
+          use Rack::ContentLength
+          use Rack::ContentType, "text/plain"
+          run(lambda do |env|
+            case env['PATH_INFO']
+            when '/'; return [ 200, {}, [ str ] ]
+            when '/304'; return [ 304, {}, [ str ] ]
+            else
+              abort 'unsupported'
+            end
+          end)
+        end
+        app(:rack, app) { listen "#{host}:#{port}" }
+      end
+      logger(Logger.new(err.path))
+    end
+    pid = mkserver(cfg)
+    s = TCPSocket.new(host, port)
+    s.write("HEAD / HTTP/1.0\r\n\r\n")
+    assert s.wait(30), "IO wait failed"
+    buf = s.read
+    assert_match %r{\r\n\r\n\z}, buf
+    s.close
+
+    s = TCPSocket.new(host, port)
+    s.write("GET /304 HTTP/1.0\r\n\r\n")
+    assert s.wait(30), "IO wait failed"
+    buf = s.read
+    assert_match %r{\r\n\r\n\z}, buf
+    assert_match %r{\b304\b}, buf
+    s.close
+  ensure
+    quit_wait(pid)
+  end
+
   def test_response_time_empty_body
     err = @err
     cfg = Yahns::Config.new