Fix request headline.

Resolves #206

lib/http/request.rb

@@ -67,7 +67,7 @@ class UnsupportedSchemeError < RequestError; end
     # :nodoc:
     def initialize(verb, uri, headers = {}, proxy = {}, body = nil, version = "1.1") # rubocop:disable ParameterLists
       @verb   = verb.to_s.downcase.to_sym
-      @uri    = HTTP::URI.parse uri
+      @uri    = HTTP::URI.parse(uri).normalize
       @scheme = @uri.scheme && @uri.scheme.to_s.downcase.to_sym
 
       fail(UnsupportedMethodError, "unknown method: #{verb}") unless METHODS.include?(@verb)
@@ -121,7 +121,8 @@ def connect_using_proxy(socket)
 
     # Compute HTTP request header for direct or proxy request
     def request_header
-      "#{verb.to_s.upcase} #{uri.normalize} HTTP/#{version}"
+      request_uri = using_proxy? ? uri : uri.omit(:scheme, :authority)
+      "#{verb.to_s.upcase} #{request_uri} HTTP/#{version}"
     end
 
     # Compute HTTP request header SSL proxy connection

spec/lib/http/request_spec.rb

@@ -1,8 +1,9 @@
 RSpec.describe HTTP::Request do
+  let(:proxy)       { {} }
   let(:headers)     { {:accept => "text/html"} }
-  let(:request_uri) { "http://example.com/" }
+  let(:request_uri) { "http://example.com/foo?bar=baz#moo" }
 
-  subject(:request) { HTTP::Request.new(:get, request_uri, headers) }
+  subject(:request) { HTTP::Request.new(:get, request_uri, headers, proxy) }
 
   it "includes HTTP::Headers::Mixin" do
     expect(described_class).to include HTTP::Headers::Mixin
@@ -136,4 +137,15 @@
     subject { request.caching }
     it { is_expected.to be_a HTTP::Request::Caching }
   end
+
+  describe "#request_header" do
+    subject { request.request_header }
+
+    it { is_expected.to eq "GET /foo?bar=baz#moo HTTP/1.1" }
+
+    context "with proxy" do
+      let(:proxy) { {:user => "user", :pass => "pass"} }
+      it { is_expected.to eq "GET http://example.com/foo?bar=baz#moo HTTP/1.1" }
+    end
+  end
 end