[CLIENT] Use default port when host and protocol are specified but no port

Author: estolfo

elasticsearch-transport/lib/elasticsearch/transport/client.rb

@@ -195,12 +195,20 @@ def __parse_host(host)
         when String
           if host =~ /^[a-z]+\:\/\//
             uri = URI.parse(host)
+
+            # Handle when port is not specified
+            if host =~ /^#{uri.scheme}:\/\/#{uri.host}$/
+              port = nil
+            else
+              port = uri.port
+            end
+
             { :scheme => uri.scheme,
               :user => uri.user,
               :password => uri.password,
               :host => uri.host,
               :path => uri.path,
-              :port => uri.port }
+              :port => port }
           else
             host, port = host.split(':')
             { :host => host,

elasticsearch-transport/spec/elasticsearch/transport/client_spec.rb

@@ -288,6 +288,19 @@
         end
       end
 
+      context 'when there is one host with a protocol and no port' do
+
+        let(:host) do
+          ['http://myhost']
+        end
+
+        it 'extracts the host' do
+          expect(hosts[0][:host]).to eq('myhost')
+          expect(hosts[0][:protocol]).to eq('http')
+          expect(hosts[0][:port]).to be(9200)
+        end
+      end
+
       context 'when there is more than one host' do
 
         let(:host) do