proxy_pass: no point in closing StringIO

It does not release memory immediately and GC can handle it
reliably, so don't waste a constant lookup + cache entry on it.

While we're at it, explain why we can't do a simpler
respond_to? check instead.

1 files changed, 3 insertions, 1 deletions

diff --git a/lib/yahns/proxy_pass.rb b/lib/yahns/proxy_pass.rb
index 7bbab51..15c9f53 100644
--- a/lib/yahns/proxy_pass.rb
+++ b/lib/yahns/proxy_pass.rb
@@ -149,8 +149,10 @@ class Yahns::ProxyPass # :nodoc:
     end
 
     def close_req_body(input)
+      # we cannot use respond_to?(:close) here since Rack::Lint::InputWrapper
+      # tries to prevent that (and hijack means all Rack specs go out the door)
       case input
-      when Yahns::TeeInput, IO, StringIO
+      when Yahns::TeeInput, IO
         input.close
       end
     end