[fix] logging behavior for query hashes

lib/excon/instrumentors/logging_instrumentor.rb

@@ -28,7 +28,7 @@ def self.instrument(name, params = {})
           info << "?"
 
           if params[:query].is_a?(Hash)
-            info << params.to_a.map{ |key,value| "#{key}=#{value}" }.join('&')
+            info << params[:query].to_a.map { |key,value| "#{key}=#{value}" }.join('&')
           else
             info << params[:query]
           end

tests/instrumentors/logging_instrumentor_tests.rb

@@ -26,6 +26,33 @@
     ]
   end
 
+  tests("connection logger with query as hash").returns(true) do
+    Excon.stub({:method => :get}, {body: 'body'}, status: 200})
+
+    log_path = "/tmp/excon_#{Time.now.to_i}.txt"
+    logger = Logger.new(log_path)
+    # omit datetime to simplify test matcher
+    logger.formatter = proc do |severity, datetime, progname, msg|
+      "#{msg}\n"
+    end
+
+    connection = Excon.new(
+      'http://127.0.0.1:9292',
+      instrumentor: Excon::LoggingInstrumentor,
+      logger: logger,
+      mock: true
+    )
+    response = connection.request(
+      method: :get,
+      path: '/logger',
+      query: {test: 'test'}
+    )
+    File.readlines(log_path)[1..2] == [
+      "request: http://127.0.0.1/logger?test=test\n",
+      "response: body\n"
+    ]
+  end
+
   Excon.stubs.clear
   env_restore
 end