rack-proxy 0.3.6 → 0.3.7

data/Gemfile

@@ -1,4 +1,6 @@
 source "http://rubygems.org"
 
+gem 'rake'
+
 # Specify your gem's dependencies in rack-proxy.gemspec
 gemspec

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    rack-proxy (0.3.4)
+    rack-proxy (0.3.7)
       rack
 
 GEM
@@ -10,11 +10,12 @@ GEM
     rack (1.2.1)
     rack-test (0.5.6)
       rack (>= 1.0)
+    rake (0.9.2.2)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
-  rack
   rack-proxy!
   rack-test
+  rake

data/Rakefile

@@ -1,3 +1,4 @@
+require 'rubygems'
 require 'bundler'
 Bundler::GemHelper.install_tasks
 

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.3.6"
+    VERSION = "0.3.7"
     
     def call(env)
       rewrite_response(perform_request(rewrite_env(env)))
@@ -49,7 +49,7 @@ module Rack
     
     def extract_http_request_headers(env)
       headers = env.reject do |k, v|
-        !(/^HTTP_[A-Z_]+$/ === k)
+        !(/^HTTP_[A-Z_]+$/ === k) || v.nil?
       end.map do |k, v|
         [reconstruct_header_name(k), v]
       end.inject(Utils::HeaderHash.new) do |hash, k_v|

data/test/net_http_hacked_test.rb

@@ -17,7 +17,7 @@ class NetHttpHackedTest < Test::Unit::TestCase
 
     assert headers.size > 0
     assert headers["content-type"] == "text/html; charset=UTF-8"
-    assert headers["content-length"].to_i > 0
+    assert !headers["date"].nil?
     
     # Body
     chunks = []

data/test/rack_proxy_test.rb

@@ -5,7 +5,6 @@ class RackProxyTest < Test::Unit::TestCase
   class TrixProxy < Rack::Proxy
     def rewrite_env(env)
       env["HTTP_HOST"] = "trix.pl"
-      
       env
     end
   end
@@ -29,4 +28,18 @@ class RackProxyTest < Test::Unit::TestCase
     header = proxy.send(:reconstruct_header_name, "HTTP_ABC_D")
     assert header == "ABC-D"
   end
+
+  def test_extract_http_request_headers
+    proxy = Rack::Proxy.new
+    env = {
+      'NOT-HTTP-HEADER' => 'test-value',
+      'HTTP_ACCEPT' => 'text/html',
+      'HTTP_CONNECTION' => nil
+    }
+
+    headers = proxy.send(:extract_http_request_headers, env)
+    assert headers.key?('ACCEPT')
+    assert !headers.key?('CONNECTION')
+    assert !headers.key?('NOT-HTTP-HEADER')
+  end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: rack-proxy
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: 
   segments: 
   - 0
   - 3
-  - 6
-  version: 0.3.6
+  - 7
+  version: 0.3.7
 platform: ruby
 authors: 
 - Jacek Becela
@@ -15,10 +15,10 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2012-01-09 00:00:00 Z
+date: 2012-02-02 00:00:00 +01:00
+default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  name: rack
   prerelease: false
   requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
@@ -30,9 +30,9 @@ dependencies:
         - 0
         version: "0"
   type: :runtime
+  name: rack
   version_requirements: *id001
 - !ruby/object:Gem::Dependency 
-  name: rack-test
   prerelease: false
   requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
@@ -44,6 +44,7 @@ dependencies:
         - 0
         version: "0"
   type: :development
+  name: rack-test
   version_requirements: *id002
 description: A Rack app that provides request/response rewriting proxy capabilities with streaming.
 email: 
@@ -69,6 +70,7 @@ files:
 - test/net_http_hacked_test.rb
 - test/rack_proxy_test.rb
 - test/test_helper.rb
+has_rdoc: true
 homepage: http://rubygems.org/gems/rack-proxy
 licenses: []
 
@@ -98,7 +100,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: rack-proxy
-rubygems_version: 1.8.10
+rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: A request/response rewriting HTTP proxy. A Rack app.