puffing-billy 0.5.1 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 44bd0fde2cbcc88437c9485b627b7a7c663036aa
-  data.tar.gz: 3568098fec618acceb6c26a8eb9791d79dbe7933
+  metadata.gz: c7dab062648498106cd36d92beb9ac60d785258e
+  data.tar.gz: 2eb64804cc5f68b5dba48b20c589140609136f7f
 SHA512:
-  metadata.gz: cb89785eee28a5663d13f83f3dc4e1eb337ecdd5e9ae6823cd3be3f9eed14ddb44c9b05762ce97bb927c2a50d2f71696b806111ebb48a20849fde3cca6425370
-  data.tar.gz: d4ce0f9d90c921c82c3313ef2e7495a28d934217778678013a4c608ec7c8027e72dbed468013e3caf155b2c0154faad9caa7d007f731385ad32cf869534e1349
+  metadata.gz: 4187b46b234b85781c70d0b7c6b627741b97618036cc28fe6f1e9129aa26623f72e2bd7b9b17b913d55427ea7b88e7afa8b0ae6852d7ebfa75ac0b690f372c37
+  data.tar.gz: ad0efae508e1016672e7b37185e4b04c5701bb22615fb40d8d1169023a4ad9e0587286ea3c8bc79394a4b21c7cc9d88ccf4aa40109fb0084a19a71e57e4bda9b

data/CHANGELOG.md

@@ -1,3 +1,10 @@
+v0.6.0, 2015-08-25
+------------------
+
+  * Fix uninitalized constant error in Minitest (#109)
+  * Add support for customizing Billy proxy host (#112)
+  * Add support for internal proxies (#118)
+
 v0.5.1, 2015-04-22
 ------------------
 

data/Gemfile.lock

@@ -1,11 +1,11 @@
 PATH
   remote: .
   specs:
-    puffing-billy (0.5.0)
+    puffing-billy (0.5.1)
       addressable
       em-http-request (~> 1.1.0)
       em-synchrony
-      eventmachine
+      eventmachine (= 1.0.4)
       eventmachine_httpserver
       http_parser.rb (~> 0.6.0)
       multi_json
@@ -13,31 +13,32 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    addressable (2.3.6)
+    addressable (2.3.8)
     builder (3.2.2)
-    capybara (2.1.0)
+    capybara (2.4.4)
       mime-types (>= 1.16)
       nokogiri (>= 1.3.3)
       rack (>= 1.0.0)
       rack-test (>= 0.5.4)
       xpath (~> 2.0)
-    capybara-webkit (1.1.1)
-      capybara (>= 2.0.2, < 2.2.0)
+    capybara-webkit (1.6.0)
+      capybara (>= 2.3.0, < 2.5.0)
       json
-    celluloid (0.15.2)
-      timers (~> 1.1.0)
-    childprocess (0.4.0)
+    childprocess (0.5.6)
       ffi (~> 1.0, >= 1.0.11)
     cliver (0.3.2)
     coderay (1.1.0)
     cookiejar (0.3.2)
-    cucumber (1.3.10)
+    cucumber (2.0.2)
       builder (>= 2.1.2)
+      cucumber-core (~> 1.2.0)
       diff-lcs (>= 1.1.3)
       gherkin (~> 2.12)
       multi_json (>= 1.7.5, < 2.0)
-      multi_test (>= 0.0.2)
-    daemons (1.1.9)
+      multi_test (>= 0.1.2)
+    cucumber-core (1.2.0)
+      gherkin (~> 2.12.0)
+    daemons (1.2.3)
     diff-lcs (1.2.5)
     em-http-request (1.1.2)
       addressable (>= 2.3.4)
@@ -47,75 +48,88 @@ GEM
       http_parser.rb (>= 0.6.0)
     em-socksify (0.3.0)
       eventmachine (>= 1.0.0.beta.4)
-    em-synchrony (1.0.3)
+    em-synchrony (1.0.4)
       eventmachine (>= 1.0.0.beta.1)
-    eventmachine (1.0.3)
+    eventmachine (1.0.4)
     eventmachine_httpserver (0.2.1)
-    faraday (0.9.0)
+    faraday (0.9.1)
       multipart-post (>= 1.2, < 3)
-    ffi (1.9.3)
-    formatador (0.2.4)
+    ffi (1.9.10)
+    formatador (0.2.5)
     gherkin (2.12.2)
       multi_json (~> 1.3)
-    guard (2.4.0)
+    guard (2.13.0)
       formatador (>= 0.2.4)
-      listen (~> 2.1)
+      listen (>= 2.7, <= 4.0)
       lumberjack (~> 1.0)
+      nenv (~> 0.1)
+      notiffany (~> 0.0)
       pry (>= 0.9.12)
+      shellany (~> 0.0)
       thor (>= 0.18.1)
     http_parser.rb (0.6.0)
-    json (1.8.1)
-    listen (2.4.1)
-      celluloid (>= 0.15.2)
+    json (1.8.3)
+    listen (3.0.3)
       rb-fsevent (>= 0.9.3)
       rb-inotify (>= 0.9)
-    lumberjack (1.0.4)
+    lumberjack (1.0.9)
     method_source (0.8.2)
-    mime-types (2.1)
-    mini_portile (0.5.2)
-    multi_json (1.8.4)
-    multi_test (0.0.3)
+    mime-types (2.6.1)
+    mini_portile (0.6.2)
+    multi_json (1.11.2)
+    multi_test (0.1.2)
     multipart-post (2.0.0)
-    nokogiri (1.6.1)
-      mini_portile (~> 0.5.0)
-    poltergeist (1.5.0)
+    nenv (0.2.0)
+    nokogiri (1.6.6.2)
+      mini_portile (~> 0.6.0)
+    notiffany (0.0.7)
+      nenv (~> 0.1)
+      shellany (~> 0.0)
+    poltergeist (1.6.0)
       capybara (~> 2.1)
       cliver (~> 0.3.1)
       multi_json (~> 1.0)
       websocket-driver (>= 0.2.0)
-    pry (0.9.12.6)
-      coderay (~> 1.0)
-      method_source (~> 0.8)
+    pry (0.10.1)
+      coderay (~> 1.1.0)
+      method_source (~> 0.8.1)
       slop (~> 3.4)
-    rack (1.5.2)
-    rack-test (0.6.2)
+    rack (1.6.4)
+    rack-test (0.6.3)
       rack (>= 1.0)
-    rb-fsevent (0.9.4)
-    rb-inotify (0.9.3)
+    rb-fsevent (0.9.5)
+    rb-inotify (0.9.5)
       ffi (>= 0.5.0)
-    rspec (2.14.1)
-      rspec-core (~> 2.14.0)
-      rspec-expectations (~> 2.14.0)
-      rspec-mocks (~> 2.14.0)
-    rspec-core (2.14.7)
-    rspec-expectations (2.14.5)
-      diff-lcs (>= 1.1.3, < 2.0)
-    rspec-mocks (2.14.5)
-    rubyzip (1.1.0)
-    selenium-webdriver (2.39.0)
-      childprocess (>= 0.2.5)
+    rspec (3.3.0)
+      rspec-core (~> 3.3.0)
+      rspec-expectations (~> 3.3.0)
+      rspec-mocks (~> 3.3.0)
+    rspec-core (3.3.2)
+      rspec-support (~> 3.3.0)
+    rspec-expectations (3.3.1)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.3.0)
+    rspec-mocks (3.3.2)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.3.0)
+    rspec-support (3.3.0)
+    rubyzip (1.1.7)
+    selenium-webdriver (2.47.1)
+      childprocess (~> 0.5)
       multi_json (~> 1.0)
       rubyzip (~> 1.0)
-      websocket (~> 1.0.4)
-    slop (3.4.7)
-    thin (1.6.1)
-      daemons (>= 1.0.9)
-      eventmachine (>= 1.0.0)
-      rack (>= 1.0.0)
-    thor (0.18.1)
-    timers (1.1.0)
-    websocket (1.0.7)
-    websocket-driver (0.3.2)
+      websocket (~> 1.0)
+    shellany (0.0.1)
+    slop (3.6.0)
+    thin (1.6.3)
+      daemons (~> 1.0, >= 1.0.9)
+      eventmachine (~> 1.0)
+      rack (~> 1.0)
+    thor (0.19.1)
+    websocket (1.2.2)
+    websocket-driver (0.6.2)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.2)
     xpath (2.0.0)
       nokogiri (~> 1.3)
 
@@ -136,3 +150,6 @@ DEPENDENCIES
   rspec
   selenium-webdriver
   thin
+
+BUNDLED WITH
+   1.10.6

data/README.md

@@ -133,9 +133,9 @@ And /^a stub for google$/ do
 end
 ```
 
-It's good practice to reset the driver after each scenario, so having an 
-`@billy` tag switches the drivers on for a given scenario. Also note that 
-stubs are reset after each step, so any usage of a stub should be in the 
+It's good practice to reset the driver after each scenario, so having an
+`@billy` tag switches the drivers on for a given scenario. Also note that
+stubs are reset after each step, so any usage of a stub should be in the
 same step that it was created in.
 
 ## Minitest Usage
@@ -194,6 +194,10 @@ Billy.configure do |c|
   c.non_successful_error_level = :warn
   c.non_whitelisted_requests_disabled = false
   c.cache_path = 'spec/req_cache/'
+  c.proxy_host = 'example.com' # defaults to localhost
+  c.proxy_port = 12345 # defaults to random
+  c.proxied_request_host = nil
+  c.proxied_request_port = 80
 end
 ```
 
@@ -216,10 +220,10 @@ generally the same as: `proxy.stub('http://myapi.com/user/')`. When you need to
 you may set this config value to false.
 
 `c.dynamic_jsonp` is used to rewrite the body of JSONP responses based on the
-callback parameter. For example, if a request to `http://example.com/foo?callback=bar` 
+callback parameter. For example, if a request to `http://example.com/foo?callback=bar`
 returns `bar({"some": "json"});` and is recorded, then a later request to
 `http://example.com/foo?callback=baz` will be a cache hit and respond with
-`baz({"some": "json"});` This is useful because most JSONP implementations 
+`baz({"some": "json"});` This is useful because most JSONP implementations
 base the callback name off of a timestamp or something else dynamic.
 
 `c.dynamic_jsonp_keys` is used to configure which parameters to ignore when
@@ -256,6 +260,11 @@ allowed, all others will throw an error with the URL attempted to be accessed.
 This is useful for debugging issues in isolated environments (ie.
 continuous integration).
 
+`c.proxy_host` and `c.proxy_port` are used for the Billy proxy itself which runs locally.
+
+`c.proxied_request_host` and `c.proxied_request_port` are used if an internal proxy
+server is required to access the internet.  Most common in larger companies.
+
 ### Cache Scopes
 
 If you need to cache different responses to the same HTTP request, you can use
@@ -294,7 +303,7 @@ context 'with a cache scope' do
   end
 
   # Or you can run a block within the context of a cache scope:
-  # Note: When using scope blocks, be sure that both the action that triggers a 
+  # Note: When using scope blocks, be sure that both the action that triggers a
   #       request and the assertion that a response has been received are within the block
   it 'can execute a block against a named cache' do
     expect(proxy.cache.scope).to eq("my_cache")

data/lib/billy/config.rb

@@ -8,9 +8,9 @@ module Billy
 
     attr_accessor :logger, :cache, :cache_request_headers, :whitelist, :path_blacklist, :ignore_params,
                   :persist_cache, :ignore_cache_port, :non_successful_cache_disabled, :non_successful_error_level,
-                  :non_whitelisted_requests_disabled, :cache_path, :proxy_port, :proxied_request_inactivity_timeout,
+                  :non_whitelisted_requests_disabled, :cache_path, :proxy_host, :proxy_port, :proxied_request_inactivity_timeout,
                   :proxied_request_connect_timeout, :dynamic_jsonp, :dynamic_jsonp_keys, :merge_cached_responses_whitelist,
-                  :strip_query_params
+                  :strip_query_params, :proxied_request_host, :proxied_request_port
 
     def initialize
       @logger = defined?(Rails) ? Rails.logger : Logger.new(STDOUT)
@@ -32,10 +32,13 @@ module Billy
       @non_successful_error_level = :warn
       @non_whitelisted_requests_disabled = false
       @cache_path = File.join(Dir.tmpdir, 'puffing-billy')
+      @proxy_host = 'localhost'
       @proxy_port = RANDOM_AVAILABLE_PORT
       @proxied_request_inactivity_timeout = 10 # defaults from https://github.com/igrigorik/em-http-request/wiki/Redirects-and-Timeouts
       @proxied_request_connect_timeout = 5
       @strip_query_params = true
+      @proxied_request_host = nil
+      @proxied_request_port = 80
     end
   end
 

data/lib/billy/handlers/proxy_handler.rb

@@ -13,10 +13,15 @@ module Billy
 
     def handle_request(method, url, headers, body)
       if handles_request?(method, url, headers, body)
-        req = EventMachine::HttpRequest.new(url,
-                                            inactivity_timeout: Billy.config.proxied_request_inactivity_timeout,
-                                            connect_timeout: Billy.config.proxied_request_connect_timeout)
+        opts = { inactivity_timeout: Billy.config.proxied_request_inactivity_timeout,
+                 connect_timeout:    Billy.config.proxied_request_connect_timeout }
 
+        if Billy.config.proxied_request_host && !bypass_internal_proxy?(url)
+          opts.merge!({ proxy: { host: Billy.config.proxied_request_host,
+                                 port: Billy.config.proxied_request_port }} )
+        end
+
+        req = EventMachine::HttpRequest.new(url, opts)
         req = req.send(method.downcase, build_request_options(headers, body))
 
         if req.error
@@ -109,5 +114,9 @@ module Billy
     def cacheable_status?(status)
       Billy.config.non_successful_cache_disabled ? successful_status?(status) : true
     end
+
+    def bypass_internal_proxy?(url)
+      url.include?('localhost') || url.include?('127.') || url.include?('.dev') || url.include?('.fin')
+    end
   end
 end

data/lib/billy/handlers/request_handler.rb

@@ -1,3 +1,5 @@
+require 'forwardable'
+
 module Billy
   class RequestHandler
     extend Forwardable

data/lib/billy/proxy.rb

@@ -27,7 +27,7 @@ module Billy
     end
 
     def host
-      'localhost'
+      Billy.config.proxy_host
     end
 
     def port

data/lib/billy/version.rb

@@ -1,3 +1,3 @@
 module Billy
-  VERSION = '0.5.1'
+  VERSION = '0.6.0'
 end

data/puffing-billy.gemspec

@@ -28,7 +28,7 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency 'pry'
   gem.add_development_dependency 'cucumber'
   gem.add_runtime_dependency 'addressable'
-  gem.add_runtime_dependency 'eventmachine'
+  gem.add_runtime_dependency 'eventmachine', '= 1.0.4'
   gem.add_runtime_dependency 'em-synchrony'
   gem.add_runtime_dependency 'em-http-request', '~> 1.1.0'
   gem.add_runtime_dependency 'eventmachine_httpserver'

data/spec/lib/billy/handlers/proxy_handler_spec.rb

@@ -168,6 +168,22 @@ describe Billy::ProxyHandler do
                                request[:headers],
                                request[:body])
       end
+
+      it 'uses the internal proxy settings defined in configuration' do
+        allow(Billy.config).to receive(:proxied_request_host).and_return('10.10.10.10')
+        allow(Billy.config).to receive(:proxied_request_port).and_return('2080')
+
+        expect(EventMachine::HttpRequest).to receive(:new).with(request[:url],
+                                                                inactivity_timeout: 10,
+                                                                connect_timeout: 5,
+                                                                proxy: { host: '10.10.10.10', port: '2080' }
+        )
+
+        subject.handle_request(request[:method],
+                               request[:url],
+                               request[:headers],
+                               request[:body])
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: puffing-billy
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.6.0
 platform: ruby
 authors:
 - Olly Smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-22 00:00:00.000000000 Z
+date: 2015-08-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -196,16 +196,16 @@ dependencies:
   name: eventmachine
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.4
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - '='
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 1.0.4
 - !ruby/object:Gem::Dependency
   name: em-synchrony
   requirement: !ruby/object:Gem::Requirement
@@ -356,7 +356,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.8
 signing_key: 
 specification_version: 4
 summary: Easy request stubs for browser tests.
@@ -376,3 +376,4 @@ test_files:
 - spec/lib/proxy_spec.rb
 - spec/spec_helper.rb
 - spec/support/test_server.rb
+has_rdoc: