Add test for basicauth+proxy case

Author: mdpye

spec/http_proxy_spec.rb

@@ -52,6 +52,23 @@
       }
     end
 
+    it "should strip basic auth from before the host in URI sent to proxy" do
+      EventMachine.run {
+
+        http = EventMachine::HttpRequest.new('http://user:pass@127.0.0.1:8090/echo_authorization_header', proxy).get
+
+        http.errback { failed(http) }
+        http.callback {
+          http.response_header.status.should == 200
+          # The test proxy server gives the requested uri back in this header
+          http.response_header['X_THE_REQUESTED_URI'].should == 'http://127.0.0.1:8090/echo_authorization_header'
+          # Ensure the basic auth was converted to a header correctly
+          http.response.should match('authorization:Basic dXNlcjpwYXNz')
+          EventMachine.stop
+        }
+      }
+    end
+
     it "should include query parameters specified in the options" do
       EventMachine.run {
         http = EventMachine::HttpRequest.new('http://127.0.0.1:8090/', proxy).get :query => { 'q' => 'test' }

spec/stallion.rb

@@ -96,6 +96,9 @@ def self.call(env)
     elsif stable.request.path_info == '/echo_content_length_from_header'
       stable.response.write "content-length:#{stable.request.env["CONTENT_LENGTH"]}"
 
+    elsif stable.request.path_info == '/echo_authorization_header'
+      stable.response.write "authorization:#{stable.request.env["HTTP_AUTHORIZATION"]}"
+
     elsif stable.request.head? && stable.request.path_info == '/'
       stable.response.status = 200