rack-reverse-proxy 0.9.1 → 0.10.0

data/spec/rack_reverse_proxy/response_builder_spec.rb

@@ -0,0 +1,37 @@
+require "spec_helper"
+
+module RackReverseProxy
+  RSpec.describe ResponseBuilder do
+    let(:options) { {} }
+    let(:uri_opt) { { :uri => "http://example.org/hello/world" } }
+    let(:request) { double("Request") }
+
+    subject(:response) do
+      ResponseBuilder.new(
+        request,
+        URI(uri_opt[:uri]),
+        options
+      ).fetch
+    end
+
+    it "is a Rack::HttpStreamingResponse" do
+      expect(response).to be_a(Rack::HttpStreamingResponse)
+    end
+
+    it "sets up read timeout" do
+      options[:timeout] = 42
+      expect(response.read_timeout).to eq(42)
+    end
+
+    it "sets up ssl when needed" do
+      uri_opt[:uri] = "https://example.org/hello/world"
+      expect(response.use_ssl).to eq(true)
+    end
+
+    it "it is possible to change ssl verify mode" do
+      mode = OpenSSL::SSL::VERIFY_NONE
+      options[:verify_mode] = mode
+      expect(response.verify_mode).to eq(mode)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,106 @@
+# Code coverage
+require "simplecov"
+SimpleCov.start
+
+require "rack/reverse_proxy"
+require "rack/test"
+require "webmock/rspec"
+
+# Patch HttpStreamingResponse to make rack-proxy compatible with webmocks
+require "support/http_streaming_response_patch"
+
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause
+# this file to always be loaded, without a need to explicitly require it in any
+# files.
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need
+# it.
+#
+# The `.rspec` file also contains a few flags that are not defaults but that
+# users commonly want.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+RSpec.configure do |config|
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    #     be_bigger_than(2).and_smaller_than(4).description
+    #     # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #     # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+  # These two settings work together to allow you to limit a spec run
+  # to individual examples or groups you care about by tagging them with
+  # `:focus` metadata. When nothing is tagged with `:focus`, all examples
+  # get run.
+  config.filter_run :focus
+  config.run_all_when_everything_filtered = true
+
+  # Allows RSpec to persist some state between runs in order to support
+  # the `--only-failures` and `--next-failure` CLI options. We recommend
+  # you configure your source control system to ignore this file.
+  config.example_status_persistence_file_path = "spec/examples.txt"
+
+  # Limits the available syntax to the non-monkey patched syntax that is
+  # recommended. For more details, see:
+  #   - http://myronmars.to/n/dev-blog/2012/06/rspecs-new-expectation-syntax
+  #   - http://www.teaisaweso.me/blog/2013/05/27/rspecs-new-message-expectation-syntax/
+  #   - http://myronmars.to/n/dev-blog/2014/05/notable-changes-in-rspec-3#new__config_option_to_disable_rspeccore_monkey_patching
+  config.disable_monkey_patching!
+
+  # This setting enables warnings. It's recommended, but in some cases may
+  # be too noisy due to issues in dependencies.
+  config.warnings = true
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    config.default_formatter = "doc"
+  end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  Kernel.srand config.seed
+
+  # User-defined configuration
+  WebMock.disable_net_connect!
+end

data/spec/support/http_streaming_response_patch.rb

@@ -0,0 +1,32 @@
+module Rack
+  ##
+  # Patch HttpStreamingResponse
+  # in order to support webmocks and still use rack-proxy
+  #
+  # Inspired by @ehlertij commits on sportngin/rack-proxy:
+  # 616574e452fa731f5427d2ff2aff6823fcf28bde
+  # d8c377f7485997b229ced23c33cfef87d3fb8693
+  # 75b446a26ceb519ddc28f38b33309e9a2799074c
+  #
+  class HttpStreamingResponse
+    def each(&block)
+      response.read_body(&block)
+    ensure
+      session.end_request_hacked unless mocking?
+    end
+
+    protected
+
+    def response
+      if mocking?
+        @response ||= session.request(@request)
+      else
+        super
+      end
+    end
+
+    def mocking?
+      defined?(WebMock) || defined?(FakeWeb)
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rack-reverse-proxy
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.10.0
 platform: ruby
 authors:
 - Jon Swope
@@ -11,80 +11,73 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-22 00:00:00.000000000 Z
+date: 2016-02-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.3'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '10.3'
-- !ruby/object:Gem::Dependency
-  name: guard-rspec
+  name: rack
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: 1.0.0
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.0
 - !ruby/object:Gem::Dependency
-  name: guard-bundler
+  name: rack-proxy
   requirement: !ruby/object:Gem::Requirement
     requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
+        version: 0.5.14
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 0.5.14
 - !ruby/object:Gem::Dependency
-  name: rack
+  name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
-  type: :runtime
+        version: '1.7'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: '1.7'
 - !ruby/object:Gem::Dependency
-  name: rack-proxy
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
-  type: :runtime
+        version: '10.3'
+  type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.5'
-description: A Rack based reverse proxy for basic needs.  Useful for testing or in
-  cases where webserver configuration is unavailable.
+        version: '10.3'
+description: |
+  A Rack based reverse proxy for basic needs.
+  Useful for testing or in cases where webserver configuration is unavailable.
 email:
 - jaswope@gmail.com
 - ehlertij@gmail.com
@@ -94,14 +87,33 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".document"
+- ".gitignore"
+- ".rspec"
+- ".rubocop.yml"
+- ".travis.yml"
+- CHANGELOG.md
+- Gemfile
 - LICENSE
 - README.md
-- lib/rack/exception.rb
+- Rakefile
 - lib/rack/reverse_proxy.rb
-- lib/rack/reverse_proxy/http_streaming_response.rb
-- lib/rack/reverse_proxy_matcher.rb
-homepage: http://github.com/waterlink/rack-reverse-proxy
-licenses: []
+- lib/rack_reverse_proxy.rb
+- lib/rack_reverse_proxy/errors.rb
+- lib/rack_reverse_proxy/middleware.rb
+- lib/rack_reverse_proxy/response_builder.rb
+- lib/rack_reverse_proxy/roundtrip.rb
+- lib/rack_reverse_proxy/rule.rb
+- lib/rack_reverse_proxy/version.rb
+- rack-reverse-proxy.gemspec
+- script/rubocop
+- spec/rack/reverse_proxy_spec.rb
+- spec/rack_reverse_proxy/response_builder_spec.rb
+- spec/spec_helper.rb
+- spec/support/http_streaming_response_patch.rb
+homepage: https://github.com/waterlink/rack-reverse-proxy
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -123,4 +135,8 @@ rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: A Simple Reverse Proxy for Rack
-test_files: []
+test_files:
+- spec/rack/reverse_proxy_spec.rb
+- spec/rack_reverse_proxy/response_builder_spec.rb
+- spec/spec_helper.rb
+- spec/support/http_streaming_response_patch.rb

data/lib/rack/exception.rb

@@ -1,31 +0,0 @@
-module Rack
-  class GenericProxyURI < Exception
-    attr_reader :url
-
-    def intialize(url)
-      @url = url
-    end
-
-    def to_s
-      %Q(Your URL "#{@url}" is too generic. Did you mean "http://#{@url}"?)
-    end
-  end
-
-  class AmbiguousProxyMatch < Exception
-    attr_reader :path, :matches
-    def initialize(path, matches)
-      @path = path
-      @matches = matches
-    end
-
-    def to_s
-      %Q(Path "#{path}" matched multiple endpoints: #{formatted_matches})
-    end
-
-    private
-
-    def formatted_matches
-      matches.map {|matcher| matcher.to_s}.join(', ')
-    end
-  end
-end

data/lib/rack/reverse_proxy/http_streaming_response.rb

@@ -1,7 +0,0 @@
-module Rack
-  class HttpStreamingResponse
-    def set_read_timeout(value)
-      self.read_timeout = value
-    end
-  end
-end

data/lib/rack/reverse_proxy_matcher.rb

@@ -1,53 +0,0 @@
-module Rack
-  class ReverseProxyMatcher
-    def initialize(matcher, url=nil, options={})
-      @default_url=url
-      @url=url
-      @options=options
-
-      if matcher.kind_of?(String)
-        @matcher = /^#{matcher.to_s}/
-      elsif matcher.respond_to?(:match)
-        @matcher = matcher
-      else
-        raise "Invalid Matcher for reverse_proxy"
-      end
-    end
-
-    attr_reader :matcher,:url, :default_url,:options
-
-    def match?(path, *args)
-      match_path(path, *args) ? true : false
-    end
-
-    def get_uri(path,env)
-      return nil if url.nil?
-      _url=(url.respond_to?(:call) ? url.call(env) : url.clone)
-      if _url =~/\$\d/
-        match_path(path).to_a.each_with_index { |m, i| _url.gsub!("$#{i.to_s}", m) }
-        URI(_url)
-      else
-        default_url.nil? ? URI.parse(_url) : URI.join(_url, path)
-      end
-    end
-
-    def to_s
-      %Q("#{matcher.to_s}" => "#{url}")
-    end
-
-    private
-    def match_path(path, *args)
-      headers = args[0]
-      rackreq = args[1]
-      arity = matcher.method(:match).arity
-      if arity == -1
-        match = matcher.match(path)
-      else
-        params = [path, (@options[:accept_headers] ? headers : nil), rackreq]
-        match = matcher.match(*params[0..(arity - 1)])
-      end
-      @url = match.url(path) if match && default_url.nil?
-      match
-    end
-  end
-end