chef-winrm 2.3.11 → 2.3.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dfe8924446478cc291356770089cd09b389d37cd281397be0ee14cf8c7f4618e
-  data.tar.gz: e7995603a7937840c0f8429d1fd721d02d00cacd11c03ee3f6ba1cf4e9e15e99
+  metadata.gz: 53783559fd592d6506745a28e356e40cb9e38272432b7d2a026f74b7e0b9cbdd
+  data.tar.gz: 7a7051fe6db091e6d94c3d8e5e4e09c52c91052f7ee704e1603813b5f4f311e8
 SHA512:
-  metadata.gz: c7b900b03ad11b098451e478d9462f0f6ecbf0da7ec0459be8700ca73583111f6740bc8a504b4a96aa2c258320b5e2110bce02481ec6f61f0fa74dbffd2c12f9
-  data.tar.gz: 52b653bef46489551e339388f5aeb2f4d0811e20dee376b96661c6b2185b71820e5d3dc3fcb197b960418ce9ccb82bbc7710b8a49a80fd89c2591d9d51a31aac
+  metadata.gz: 046a737f2af6c0e7a78fd5b66f5473490099fd39971a7841628cbd22ed326f6b4415c417e5c9495d89a9f7dc5d8b6808c368c0dd8ff3760f7c88237f57576ccf
+  data.tar.gz: efebd000fa453e153ce17b2141f7e806b54b0736c530f8987467ee5d4e4fd1bef1dc0d9374aaeb46e1ec72573a3525b486fb9fd4d65c8ed813b7796304c5e792

data/lib/chef-winrm/connection.rb

@@ -12,11 +12,11 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require_relative 'connection_opts'
-require_relative 'http/transport_factory'
-require_relative 'shells/shell_factory'
-require_relative 'wsmv/wql_query'
-require_relative 'wsmv/wql_pull'
+require_relative "connection_opts"
+require_relative "http/transport_factory"
+require_relative "shells/shell_factory"
+require_relative "wsmv/wql_query"
+require_relative "wsmv/wql_pull"
 
 module WinRM
   # WinRM connection used to establish a session with the remote WinRM service.
@@ -53,7 +53,7 @@ module WinRM
     # @param namespace [String] namespace for query - default is root/cimv2/*
     # @return [Hash] Hash representation of wql query response (Hash is empty if a block is given)
     # @yeild [type, item] Yields the time name and item for every item
-    def run_wql(wql, namespace = 'root/cimv2/*', &block)
+    def run_wql(wql, namespace = "root/cimv2/*", &block)
       query = WinRM::WSMV::WqlQuery.new(transport, @connection_opts, wql, namespace)
       query.process_response(transport.send_request(query.build), &block)
     end

data/lib/chef-winrm/connection_opts.rb

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'securerandom'
+require "securerandom" unless defined?(SecureRandom)
 
 module WinRM
   # WinRM connection options, provides defaults and validation.
@@ -20,10 +20,10 @@ module WinRM
     DEFAULT_OPERATION_TIMEOUT = 60
     DEFAULT_RECEIVE_TIMEOUT = DEFAULT_OPERATION_TIMEOUT + 10
     DEFAULT_MAX_ENV_SIZE = 153600
-    DEFAULT_LOCALE = 'en-US'.freeze
+    DEFAULT_LOCALE = "en-US".freeze
     DEFAULT_RETRY_DELAY = 10
     DEFAULT_RETRY_LIMIT = 3
-    DEFAULT_USER_AGENT = 'Ruby WinRM Client'.freeze
+    DEFAULT_USER_AGENT = "Ruby WinRM Client".freeze
 
     class << self
       def create_with_defaults(overrides)
@@ -65,13 +65,13 @@ module WinRM
     private
 
     def validate_required_fields
-      raise 'endpoint is a required option' unless self[:endpoint]
+      raise "endpoint is a required option" unless self[:endpoint]
 
       if self[:client_cert]
-        raise 'path to client key is required' unless self[:client_key]
+        raise "path to client key is required" unless self[:client_key]
       else
-        raise 'user is a required option' unless self[:user]
-        raise 'password is a required option' unless self[:password]
+        raise "user is a required option" unless self[:user]
+        raise "password is a required option" unless self[:password]
       end
     end
 

data/lib/chef-winrm/exceptions.rb

@@ -33,7 +33,7 @@ module WinRM
       @invalid_transport = invalid_transport
       super(
         "Invalid transport '#{invalid_transport}' specified" \
-        ", expected: #{valid_transports.join(', ')}."
+        ", expected: #{valid_transports.join(", ")}."
       )
     end
   end

data/lib/chef-winrm/http/response_handler.rb

@@ -12,8 +12,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'rexml/document' unless defined?(REXML::Document)
-require_relative '../wsmv/soap'
+require "rexml/document" unless defined?(REXML::Document)
+require_relative "../wsmv/soap"
 
 module WinRM
   # Handles the raw WinRM HTTP response. Returns the body as an XML doc
@@ -70,7 +70,7 @@ module WinRM
         soap_errors,
         "//*[local-name() = 'WSManFault']"
       )
-      raise WinRMWSManFault.new(fault.to_s, fault.attributes['Code']) unless fault.nil?
+      raise WinRMWSManFault.new(fault.to_s, fault.attributes["Code"]) unless fault.nil?
     end
 
     def raise_if_wmi_error

data/lib/chef-winrm/http/transport.rb

@@ -12,8 +12,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'httpclient'
-require_relative 'response_handler'
+require "httpclient"
+require_relative "response_handler"
 
 module WinRM
   module HTTP
@@ -39,11 +39,11 @@ module WinRM
       def send_request(message)
         ssl_peer_fingerprint_verification!
         log_soap_message(message)
-        hdr = { 'Content-Type' => 'application/soap+xml;charset=UTF-8',
-                'Content-Length' => message.bytesize }
+        hdr = { "Content-Type" => "application/soap+xml;charset=UTF-8",
+                "Content-Length" => message.bytesize }
         # We need to add this header if using Client Certificate authentication
         unless @httpcli.ssl_config.client_cert.nil?
-          hdr['Authorization'] = 'http://schemas.dmtf.org/wbem/wsman/1/wsman/secprofile/https/mutual'
+          hdr["Authorization"] = "http://schemas.dmtf.org/wbem/wsman/1/wsman/secprofile/https/mutual"
         end
 
         resp = @httpcli.post(@endpoint, message, hdr)
@@ -55,13 +55,13 @@ module WinRM
 
       # We'll need this to force basic authentication if desired
       def basic_auth_only!
-        auths = @httpcli.www_auth.instance_variable_get('@authenticator')
+        auths = @httpcli.www_auth.instance_variable_get("@authenticator")
         auths.delete_if { |i| i.scheme !~ /basic/i }
       end
 
       # Disable SSPI Auth
       def no_sspi_auth!
-        auths = @httpcli.www_auth.instance_variable_get('@authenticator')
+        auths = @httpcli.www_auth.instance_variable_get("@authenticator")
         auths.delete_if { |i| i.is_a? HTTPClient::SSPINegotiateAuth }
       end
 
@@ -109,14 +109,14 @@ module WinRM
 
       protected
 
-      def body(message, length, type = 'application/HTTP-SPNEGO-session-encrypted')
+      def body(message, length, type = "application/HTTP-SPNEGO-session-encrypted")
         [
-          '--Encrypted Boundary',
+          "--Encrypted Boundary",
           "Content-Type: #{type}",
           "OriginalContent: type=application/soap+xml;charset=UTF-8;Length=#{length}",
-          '--Encrypted Boundary',
-          'Content-Type: application/octet-stream',
-          "#{message}--Encrypted Boundary--"
+          "--Encrypted Boundary",
+          "Content-Type: application/octet-stream",
+          "#{message}--Encrypted Boundary--",
         ].join("\r\n").concat("\r\n")
       end
 
@@ -148,10 +148,10 @@ module WinRM
     class HttpNegotiate < HttpTransport
       def initialize(endpoint, user, pass, opts)
         super(endpoint, opts)
-        require 'rubyntlm'
+        require "rubyntlm"
         no_sspi_auth!
 
-        user_parts = user.split('\\')
+        user_parts = user.split("\\")
         if user_parts.length > 1
           opts[:domain] = user_parts[0]
           user = user_parts[1]
@@ -171,8 +171,8 @@ module WinRM
         log_soap_message(message)
 
         hdr = {
-          'Content-Type' => 'multipart/encrypted;'\
-            'protocol="application/HTTP-SPNEGO-session-encrypted";boundary="Encrypted Boundary"'
+          "Content-Type" => "multipart/encrypted;" \
+            'protocol="application/HTTP-SPNEGO-session-encrypted";boundary="Encrypted Boundary"',
         }
 
         resp = @httpcli.post(@endpoint, body(seal(message), message.bytesize), hdr)
@@ -199,29 +199,29 @@ module WinRM
 
       def winrm_decrypt(resp)
         # OMI server doesn't always respond to encrypted messages with encrypted responses over SSL
-        return resp.body if resp.header['Content-Type'].first =~ %r{\Aapplication\/soap\+xml}i
-        return '' if resp.body.empty?
+        return resp.body if resp.header["Content-Type"].first =~ %r{\Aapplication\/soap\+xml}i
+        return "" if resp.body.empty?
 
-        str = resp.body.force_encoding('BINARY')
+        str = resp.body.force_encoding("BINARY")
         str.sub!(%r{^.*Content-Type: application\/octet-stream\r\n(.*)--Encrypted.*$}m, '\1')
 
         signature = str[4..19]
         message = @ntlmcli.session.unseal_message str[20..-1]
         return message if @ntlmcli.session.verify_signature(signature, message)
 
-        raise WinRMHTTPTransportError, 'Could not decrypt NTLM message.'
+        raise WinRMHTTPTransportError, "Could not decrypt NTLM message."
       end
 
       def issue_challenge_response(negotiate)
         auth_header = {
-          'Authorization' => "Negotiate #{negotiate.encode64}",
-          'Content-Type' => 'application/soap+xml;charset=UTF-8'
+          "Authorization" => "Negotiate #{negotiate.encode64}",
+          "Content-Type" => "application/soap+xml;charset=UTF-8",
         }
 
         # OMI Server on Linux requires an empty payload with the new auth header to proceed
         # because the config check for max payload size will otherwise break the auth handshake
         # given the OMI server does not support that check
-        @httpcli.post(@endpoint, '', auth_header)
+        @httpcli.post(@endpoint, "", auth_header)
 
         # return an empty hash of headers for subsequent requests to use
         {}
@@ -231,13 +231,13 @@ module WinRM
         @logger.debug "Initializing Negotiate for #{@endpoint}"
         auth1 = @ntlmcli.init_context
         hdr = {
-          'Authorization' => "Negotiate #{auth1.encode64}",
-          'Content-Type' => 'application/soap+xml;charset=UTF-8'
+          "Authorization" => "Negotiate #{auth1.encode64}",
+          "Content-Type" => "application/soap+xml;charset=UTF-8",
         }
-        @logger.debug 'Sending HTTP POST for Negotiate Authentication'
-        r = @httpcli.post(@endpoint, '', hdr)
+        @logger.debug "Sending HTTP POST for Negotiate Authentication"
+        r = @httpcli.post(@endpoint, "", hdr)
         verify_ssl_fingerprint(r.peer_cert)
-        auth_header = r.header['WWW-Authenticate'].pop
+        auth_header = r.header["WWW-Authenticate"].pop
         unless auth_header
           msg = "Unable to parse authorization header. Headers: #{r.headers}\r\nBody: #{r.body}"
           raise WinRMHTTPTransportError.new(msg, r.status_code)
@@ -251,7 +251,7 @@ module WinRM
         if response.peer_cert.nil?
           nil
         else
-          cert = if RUBY_PLATFORM == 'java'
+          cert = if RUBY_PLATFORM == "java"
                    OpenSSL::X509::Certificate.new(response.peer_cert.cert.getEncoded)
                  else
                    response.peer_cert
@@ -279,7 +279,7 @@ module WinRM
       def initialize(endpoint, client_cert, client_key, key_pass, opts)
         super(endpoint, opts)
         @httpcli.ssl_config.set_client_cert_file(client_cert, client_key, key_pass)
-        @httpcli.www_auth.instance_variable_set('@authenticator', [])
+        @httpcli.www_auth.instance_variable_set("@authenticator", [])
         no_ssl_peer_verification! if opts[:no_ssl_peer_verification]
         @ssl_peer_fingerprint = opts[:ssl_peer_fingerprint]
         @httpcli.ssl_config.set_trust_ca(opts[:ca_trust_path]) if opts[:ca_trust_path]
@@ -293,13 +293,13 @@ module WinRM
       # @param [String] realm the Kerberos realm we are authenticating to
       # @param [String<optional>] service the service name, default is HTTP
       def initialize(endpoint, realm, opts, service = nil)
-        require 'gssapi'
-        require 'gssapi/extensions'
+        require "gssapi"
+        require "gssapi/extensions"
 
         super(endpoint, opts)
         # Remove the GSSAPI auth from HTTPClient because we are doing our own thing
         no_sspi_auth!
-        service ||= 'HTTP'
+        service ||= "HTTP"
         @service = "#{service}/#{@endpoint.host}@#{realm}"
         no_ssl_peer_verification! if opts[:no_ssl_peer_verification]
         init_krb
@@ -314,7 +314,7 @@ module WinRM
         resp = send_kerberos_request(message)
 
         if resp.status == 401
-          @logger.debug 'Got 401 - reinitializing Kerberos and retrying one more time'
+          @logger.debug "Got 401 - reinitializing Kerberos and retrying one more time"
           init_krb
           resp = send_kerberos_request(message)
         end
@@ -336,14 +336,14 @@ module WinRM
         pad_len, emsg = winrm_encrypt(message)
         req_length = original_length + pad_len
         hdr = {
-          'Connection' => 'Keep-Alive',
-          'Content-Type' => 'multipart/encrypted;' \
-            'protocol="application/HTTP-Kerberos-session-encrypted";boundary="Encrypted Boundary"'
+          "Connection" => "Keep-Alive",
+          "Content-Type" => "multipart/encrypted;" \
+            'protocol="application/HTTP-Kerberos-session-encrypted";boundary="Encrypted Boundary"',
         }
 
         resp = @httpcli.post(
           @endpoint,
-          body(emsg, req_length, 'application/HTTP-Kerberos-session-encrypted'),
+          body(emsg, req_length, "application/HTTP-Kerberos-session-encrypted"),
           hdr
         )
         log_soap_message(resp.http_body.content)
@@ -357,13 +357,13 @@ module WinRM
         auth = Base64.strict_encode64 token
 
         hdr = {
-          'Authorization' => "Kerberos #{auth}",
-          'Connection' => 'Keep-Alive',
-          'Content-Type' => 'application/soap+xml;charset=UTF-8'
+          "Authorization" => "Kerberos #{auth}",
+          "Connection" => "Keep-Alive",
+          "Content-Type" => "application/soap+xml;charset=UTF-8",
         }
-        @logger.debug 'Sending HTTP POST for Kerberos Authentication'
-        r = @httpcli.post(@endpoint, '', hdr)
-        itok = r.header['WWW-Authenticate'].pop
+        @logger.debug "Sending HTTP POST for Kerberos Authentication"
+        r = @httpcli.post(@endpoint, "", hdr)
+        itok = r.header["WWW-Authenticate"].pop
         itok = itok.split.last
         itok = Base64.strict_decode64(itok)
         @gsscli.init_context(itok)
@@ -407,7 +407,7 @@ module WinRM
           iov_cnt
         )
 
-        token = [iov0[:buffer].length].pack('L')
+        token = [iov0[:buffer].length].pack("L")
         token += iov0[:buffer].value
         token += iov1[:buffer].value
         pad_len = iov2[:buffer].length
@@ -435,10 +435,10 @@ module WinRM
         )
         iov2[:type] = GSSAPI::LibGSSAPI::GSS_IOV_BUFFER_TYPE_DATA
 
-        str.force_encoding('BINARY')
+        str.force_encoding("BINARY")
         str.sub!(%r{^.*Content-Type: application\/octet-stream\r\n(.*)--Encrypted.*$}m, '\1')
 
-        len = str.unpack('L').first
+        len = str.unpack("L").first
         iov_data = str.unpack("La#{len}a*")
         iov0[:buffer].value = iov_data[1]
         iov1[:buffer].value = iov_data[2]

data/lib/chef-winrm/http/transport_factory.rb

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require_relative 'transport'
+require_relative "transport"
 
 module WinRM
   module HTTP
@@ -46,7 +46,7 @@ module WinRM
           HTTP::BasicAuthSSL.new(opts[:endpoint], opts[:user], opts[:password], opts)
         elsif opts[:client_cert]
           HTTP::ClientCertAuthSSL.new(opts[:endpoint], opts[:client_cert],
-                                      opts[:client_key], opts[:key_pass], opts)
+            opts[:client_key], opts[:key_pass], opts)
         else
           HTTP::HttpNegotiate.new(opts[:endpoint], opts[:user], opts[:password], opts)
         end
@@ -54,8 +54,8 @@ module WinRM
 
       def validate_transport!(transport)
         valid = private_methods
-                .select { |m| m.to_s.start_with?('init_') && m.to_s.end_with?('_transport') }
-                .map { |tm| tm.to_s.split('_')[1] }
+          .select { |m| m.to_s.start_with?("init_") && m.to_s.end_with?("_transport") }
+          .map { |tm| tm.to_s.split("_")[1] }
 
         raise WinRM::InvalidTransportError.new(transport, valid) unless valid.include?(transport.to_s)
       end

data/lib/chef-winrm/psrp/fragment.rb

@@ -18,32 +18,33 @@ module WinRM
   module PSRP
     # PowerShell Remoting Protocol message fragment.
     class Fragment
+
+      attr_reader :psrp_object_id, :fragment_id, :end_fragment, :start_fragment, :blob
+
       # Creates a new PSRP message fragment
-      # @param object_id [Integer] The id of the fragmented message.
+      # @param psrp_object_id [Integer] The id of the fragmented message.
       # @param blob [Array] Array of fragmented bytes.
       # @param fragment_id [Integer] The id of this fragment
       # @param start_fragment [Boolean] If the fragment is the first fragment
       # @param end_fragment [Boolean] If the fragment is the last fragment
-      def initialize(object_id, blob, fragment_id = 0, start_fragment = true, end_fragment = true)
-        @object_id = object_id
+      def initialize(psrp_object_id, blob, fragment_id = 0, start_fragment = true, end_fragment = true)
+        @psrp_object_id = psrp_object_id
         @blob = blob
         @fragment_id = fragment_id
         @start_fragment = start_fragment
         @end_fragment = end_fragment
       end
 
-      attr_reader :object_id, :fragment_id, :end_fragment, :start_fragment, :blob
-
       # Returns the raw PSRP message bytes ready for transfer to Windows inside a
       # WinRM message.
       # @return [Array<Byte>] Unencoded raw byte array of the PSRP message.
       def bytes
         [
-          int64be(object_id),
+          int64be(psrp_object_id),
           int64be(fragment_id),
           end_start_fragment,
           int16be(blob.length),
-          blob
+          blob,
         ].flatten
       end
 
@@ -57,11 +58,11 @@ module WinRM
       end
 
       def int64be(int64)
-        [int64 >> 32, int64 & 0x00000000ffffffff].pack('N2').unpack('C8')
+        [int64 >> 32, int64 & 0x00000000ffffffff].pack("N2").unpack("C8")
       end
 
       def int16be(int16)
-        [int16].pack('N').unpack('C4')
+        [int16].pack("N").unpack("C4")
       end
     end
   end

data/lib/chef-winrm/psrp/message.rb

@@ -13,8 +13,8 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require_relative 'uuid'
-require_relative 'message_data'
+require_relative "uuid"
+require_relative "message_data"
 
 module WinRM
   # PowerShell Remoting Protcol module
@@ -62,7 +62,7 @@ module WinRM
         progress_record: 0x00041010,
         information_record: 0x00041011,
         pipeline_host_call: 0x00041100,
-        pipeline_host_response: 0x00041101
+        pipeline_host_response: 0x00041101,
       }.freeze
 
       # Creates a new PSRP message instance
@@ -79,7 +79,7 @@ module WinRM
         pipeline_id = nil,
         destination = SERVER_DESTINATION
       )
-        raise 'invalid message type' unless MESSAGE_TYPES.value?(type)
+        raise "invalid message type" unless MESSAGE_TYPES.value?(type)
 
         @data = data
         @destination = destination
@@ -100,7 +100,7 @@ module WinRM
           uuid_to_windows_guid_bytes(runspace_pool_id),
           uuid_to_windows_guid_bytes(pipeline_id),
           byte_order_mark,
-          data_bytes
+          data_bytes,
         ].flatten
       end
 
@@ -117,11 +117,11 @@ module WinRM
       end
 
       def data_bytes
-        @data_bytes ||= data.force_encoding('utf-8').bytes
+        @data_bytes ||= data.force_encoding("utf-8").bytes
       end
 
       def int16le(int16)
-        [int16].pack('N').unpack('C4').reverse
+        [int16].pack("N").unpack("C4").reverse
       end
     end
   end

data/lib/chef-winrm/psrp/message_data/base.rb

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'nori'
+require "nori"
 
 module WinRM
   # PowerShell Remoting Protcol module

data/lib/chef-winrm/psrp/message_data/error_record.rb

@@ -18,23 +18,23 @@ module WinRM
       # error record message type
       class ErrorRecord < Base
         def exception
-          @exception ||= property_hash('Exception')
+          @exception ||= property_hash("Exception")
         end
 
         def fully_qualified_error_id
-          @fully_qualified_error_id ||= string_prop('FullyQualifiedErrorId')
+          @fully_qualified_error_id ||= string_prop("FullyQualifiedErrorId")
         end
 
         def invocation_info
-          @invocation_info ||= property_hash('InvocationInfo')
+          @invocation_info ||= property_hash("InvocationInfo")
         end
 
         def error_category_message
-          @error_category_message ||= string_prop('ErrorCategory_Message')
+          @error_category_message ||= string_prop("ErrorCategory_Message")
         end
 
         def error_details_script_stack_trace
-          @error_details_script_stack_trace ||= string_prop('ErrorDetails_ScriptStackTrace')
+          @error_details_script_stack_trace ||= string_prop("ErrorDetails_ScriptStackTrace")
         end
 
         def doc
@@ -51,16 +51,16 @@ module WinRM
           return {} if prop_nodes.nil?
 
           prop_nodes.elements.each_with_object({}) do |node, props|
-            name = node.attributes['N']
+            name = node.attributes["N"]
             props[underscore(name).to_sym] = node.text if node.text
           end
         end
 
         def underscore(camel)
-          camel.gsub(/::/, '/')
-               .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
-               .gsub(/([a-z\d])([A-Z])/, '\1_\2')
-               .tr('-', '_').downcase
+          camel.gsub("::", "/")
+            .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
+            .gsub(/([a-z\d])([A-Z])/, '\1_\2')
+            .tr("-", "_").downcase
         end
       end
     end

data/lib/chef-winrm/psrp/message_data/pipeline_output.rb

@@ -12,7 +12,7 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'rexml/document' unless defined?(REXML::Document)
+require "rexml/document" unless defined?(REXML::Document)
 
 module WinRM
   module PSRP
@@ -20,19 +20,19 @@ module WinRM
       # Handles decoding a raw powershell output response
       class PipelineOutput < Base
         def output
-          extract_out_string(remove_bom(raw.force_encoding('utf-8')))
+          extract_out_string(remove_bom(raw.force_encoding("utf-8")))
         end
 
         private
 
         def extract_out_string(text)
           doc = REXML::Document.new(text)
-          doc.root.get_elements('//S').map do |node|
-            text = ''
+          doc.root.get_elements("//S").map do |node|
+            text = ""
             if node.text
               text << node.text.gsub(/(_x\h\h\h\h_)+/) do |match|
                 match.scan(/_x(\h\h\h\h)_/).flatten.map(&:hex)
-                     .pack('S*').force_encoding('utf-16le').encode('utf-8')
+                  .pack("S*").force_encoding("utf-16le").encode("utf-8")
               end.chomp
             end
             text << "\r\n"
@@ -40,7 +40,7 @@ module WinRM
         end
 
         def remove_bom(text)
-          text.sub("\xEF\xBB\xBF", '')
+          text.sub("\xEF\xBB\xBF", "")
         end
       end
     end

data/lib/chef-winrm/psrp/message_data/session_capability.rb

@@ -18,15 +18,15 @@ module WinRM
       # session capability message type
       class SessionCapability < Base
         def protocol_version
-          clixml[:version].select { |v| v.attributes['N'] == 'protocolversion' }.first
+          clixml[:version].select { |v| v.attributes["N"] == "protocolversion" }.first
         end
 
         def ps_version
-          clixml[:version].select { |v| v.attributes['N'] == 'PSVersion' }.first
+          clixml[:version].select { |v| v.attributes["N"] == "PSVersion" }.first
         end
 
         def serialization_version
-          clixml[:version].select { |v| v.attributes['N'] == 'SerializationVersion' }.first
+          clixml[:version].select { |v| v.attributes["N"] == "SerializationVersion" }.first
         end
       end
     end

data/lib/chef-winrm/psrp/message_data.rb

@@ -12,14 +12,14 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require_relative 'message_data/base'
-require_relative 'message_data/error_record'
-require_relative 'message_data/pipeline_output'
-require_relative 'message_data/pipeline_host_call'
-require_relative 'message_data/pipeline_state'
-require_relative 'message_data/runspacepool_host_call'
-require_relative 'message_data/runspacepool_state'
-require_relative 'message_data/session_capability'
+require_relative "message_data/base"
+require_relative "message_data/error_record"
+require_relative "message_data/pipeline_output"
+require_relative "message_data/pipeline_host_call"
+require_relative "message_data/pipeline_state"
+require_relative "message_data/runspacepool_host_call"
+require_relative "message_data/runspacepool_state"
+require_relative "message_data/session_capability"
 
 module WinRM
   # PowerShell Remoting Protcol module
@@ -33,7 +33,7 @@ module WinRM
       end
 
       def self.camelize(underscore)
-        underscore.split('_').collect(&:capitalize).join
+        underscore.split("_").collect(&:capitalize).join
       end
     end
   end