Merge branch 'master' of github.com:sprsquish/vertica

Author: wvanbergen

lib/vertica/connection.rb

@@ -129,6 +129,11 @@ def copy(sql, source = nil, &block)
     end
     return job.run
   end
+
+  def inspect
+    safe_options = @options.reject{ |name, _| name == :password }
+    "#<Vertica::Connection:#{object_id} @parameters=#{@parameters.inspect} @backend_pid=#{@backend_pid}, @backend_key=#{@backend_key}, @transaction_status=#{@transaction_status}, @socket=#{@socket}, @options=#{safe_options.inspect}, @row_style=#{@row_style}>"
+  end
 
   protected
 

test/functional/connection_test.rb

@@ -68,4 +68,10 @@ def test_new_with_error_response
       Vertica::Connection.new(TEST_CONNECTION_HASH.merge('database' => 'nonexistant_db'))
     end
   end
+
+  def test_connection_inspect_should_not_print_password
+    @connection = Vertica::Connection.new(TEST_CONNECTION_HASH)
+    inspected_string = @connection.inspect
+    assert_no_match /:password=>#{TEST_CONNECTION_HASH[:password]}/, inspected_string
+  end
 end