Merge pull request #171 from crepetl/fixes

winrm: hide password
inspec · Mar 27, 2017 · e10d65b · e10d65b
2 parents 4a1ba1e + b71c554
commit e10d65b
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/train/transports/winrm_connection.rb b/lib/train/transports/winrm_connection.rb
@@ -27,7 +27,7 @@ class Train::Transports::WinRM
   # host such as executing commands, transferring files, etc.
   #
   # @author Fletcher Nichol <fnichol@nichol.ca>
-  class Connection < BaseConnection
+  class Connection < BaseConnection # rubocop:disable Metrics/ClassLength
     attr_reader :hostname
     def initialize(options)
       super(options)
@@ -183,7 +183,9 @@ def session(retry_options = {})
     #
     # @api private
     def to_s
-      "<#{options.inspect}>"
+      options_to_print = @options.clone
+      options_to_print[:password] = '<hidden>' if options_to_print.key?(:password)
+      "#{@username}@#{@hostname}<#{options_to_print.inspect}>"
     end
 
     class OS < OSCommon