rack-proxy 0.8.2 → 0.8.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 40c24e4b559fc7669c3b675184af8d7f0b9ba693d3a7b41e6433274c16429adc
-  data.tar.gz: 6f8790c5c391e102275c11225a869e2572d109f2e52e4783cdc5cbbb50733579
+  metadata.gz: 47505271ab54cee7d324c7fdd22c22ba7edae1b99146ee081913668925ff61f4
+  data.tar.gz: 74afa4cf47b33eba1e12a8a4b5f8b499237fa6a6fabacbb46c0bc820e627b07b
 SHA512:
-  metadata.gz: 2885f787b2f08c712c99b938218749d7f5d35348588290414f058c446104e10ba80b7f0b439ea60c3fc4daea1a0b0f991c92e853a3a9de4ef238d5f864d85d20
-  data.tar.gz: 3f70b5ab5eee94517d10e82f72f517a7e4cd3fec346ba78142ed6d93bd34bb7d175f6e857c037c7b0beffd710889651f15d107920e38da7208bdc132f246f712
+  metadata.gz: 0ea871048c9cd7ceff7b13c7e59caad7d393947a9909e5372031cc51c85879d2e297b57c6245ea56dc7c104da3c5ed3bc5688e2ce7a8c006f68cba52c0ee1696
+  data.tar.gz: 1c3ee10747af4bc06e9a270ec21d1854f1112b82127940a198e075ee044c7be6e6c43847b83d5cfd6facfadd90a32e4260a2e8731665ced34d46ec7ffd2febc1

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rack-proxy (0.8.2)
+    rack-proxy (0.8.3)
       rack
 
 GEM

data/lib/rack/proxy.rb

@@ -5,7 +5,7 @@ module Rack
 
   # Subclass and bring your own #rewrite_request and #rewrite_response
   class Proxy
-    VERSION = "0.8.2".freeze
+    VERSION = "0.8.3".freeze
 
     HOP_BY_HOP_HEADERS = {
       'connection' => true,
@@ -128,7 +128,7 @@ module Rack
         target_request.body_stream    = source_request.body
         target_request.content_length = source_request.content_length.to_i
         target_request.content_type   = source_request.content_type if source_request.content_type
-        target_request.body_stream.rewind
+        target_request.body_stream.rewind if target_request.body_stream.respond_to?(:rewind)
       end
 
       # Use basic auth if we have to

data/test/rack_proxy_test.rb

@@ -120,6 +120,42 @@ class RackProxyTest < Test::Unit::TestCase
     end
   end
 
+  # An input stream that deliberately omits #rewind, mimicking
+  # Rackup::Handler::WEBrick::Input under Rack 3 (the Rack 3 SPEC no longer
+  # requires the input stream to respond to #rewind, only gets/each/read).
+  class NonRewindableInput
+    def initialize(string)
+      @io = StringIO.new(string)
+    end
+
+    def read(*args) = @io.read(*args)
+    def gets(*args) = @io.gets(*args)
+    def each(&block) = @io.each(&block)
+    # intentionally NO #rewind
+  end
+
+  # Issue #128: a non-rewindable body stream must not raise (it used to blow up
+  # with NoMethodError on #rewind, surfacing confusingly as a 500). The body
+  # must still be forwarded intact, since it is never read before Net::HTTP sends it.
+  def test_non_rewindable_body_is_forwarded_without_raising
+    with_webrick_proxy(streaming: false) do |port, proxy|
+      proxy.host = "127.0.0.1:#{port}"
+
+      body = NonRewindableInput.new("hello=world")
+      assert !body.respond_to?(:rewind), "fixture must not be rewindable to exercise the guard"
+
+      env = Rack::MockRequest.env_for("/echo-body", method: "POST")
+      env["rack.input"]     = body
+      env["CONTENT_LENGTH"] = "hello=world".bytesize.to_s
+      env["CONTENT_TYPE"]   = "text/plain"
+
+      status, _headers, response = nil
+      assert_nothing_raised { status, _headers, response = proxy.call(env) }
+      assert_equal 200, status.to_i
+      assert_equal "hello=world", response.to_a.join, "body must be forwarded intact"
+    end
+  end
+
   def test_response_header_included_Hop_by_hop
     app({:streaming => true}).host = 'mockapi.io'
     get 'https://example.com/oauth2/token/info?access_token=123'
@@ -308,6 +344,7 @@ class RackProxyTest < Test::Unit::TestCase
       res['x-custom'] = 'value-here'
       res.body = 'ok'
     end
+    server.mount_proc('/echo-body')    { |req, res| res.body = req.body.to_s }
     Thread.new { server.start }
     port = server.config[:Port]
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-proxy
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.3
 platform: ruby
 authors:
 - Jacek Becela