train-core 3.12.7 → 3.13.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: aec52fd6612726e12c5f2cabae1a8f45eea7602c87dc2e7d230f579d16f0ac5d
-  data.tar.gz: e19c22f689bfb0cc0d4ded96f808cc68976866bf934b53d8bca293036532a192
+  metadata.gz: 8c6ced29649aad99b00f786b6cbef4777bd154195b676fea83eb16cb4fa1961f
+  data.tar.gz: 9c75c2a466e43432fa793a1342fa509b660750ba42350fcada2ec1953d001168
 SHA512:
-  metadata.gz: b2ee00b9ab8590acb2a1c87bbbbc653141f276a2b8be5a306f7ba497b15f5010d297f9f06fadd62d5573e01de70eb967800aa529b6507d5755ce86df82ed0e8b
-  data.tar.gz: 26dbb3b68b1ec9697d20b129b64fa0eab4421b7a717b21ac234bc4d32756783f60a085221a5b70e11281d5671a15359c70c4e903a72e26118ff69805c75949e5
+  metadata.gz: 57e3ceede17d733f9f3705cc07330a3a04b1facb8ff304ffc69a1f6cc4c1b3d0441edb43e0a13f13610071befc83da1f9fef861c25452d2174f9ba59ce325053
+  data.tar.gz: 36315a4aaac156242353457b9f7a0dbf2e8c35905f71308cfd7a3986f0bb7cef0cdea66b6ae6e5cca9960acbe38ab8a071b5d9d448b13b30f690e9e521d4c18f

data/lib/train/platforms/detect/helpers/os_common.rb

@@ -8,7 +8,7 @@ module Train::Platforms::Detect::Helpers
     include Train::Platforms::Detect::Helpers::Windows
 
     def ruby_host_os(regex)
-      ::RbConfig::CONFIG["host_os"] =~ regex
+      regex.match?(::RbConfig::CONFIG["host_os"])
     end
 
     def winrm?
@@ -35,8 +35,9 @@ module Train::Platforms::Detect::Helpers
 
     def command_output(cmd)
       res = @backend.run_command(cmd)
-      stdout = res.stdout
-      stderr = res.stderr
+      # To suppress warning: literal string will be frozen in the future
+      stdout = String.new(res.stdout)
+      stderr = String.new(res.stderr)
       # When you try to execute command using ssh connection as root user and you have provided ssh user identity file
       # it gives standard output to login as authorized user other than root. To show this standard output as an error
       # to user we are matching the string of stdout and raising the error here so that user gets exact information.

data/lib/train/platforms/detect/helpers/os_windows.rb

@@ -25,6 +25,9 @@ module Train::Platforms::Detect::Helpers
         @platform[:name] = "Windows #{@platform[:release]}"
       end
 
+      # `Get-CimInstance` is a PowerShell-specific command and is not available in Command Prompt.
+      # Since the logic has always relied on `read_wmic` at this point, we are skipping the conditional check for `wmic_available?`
+      # and directly invoking `read_wmic` to maintain the existing behavior.
       read_wmic
       true
     end
@@ -42,7 +45,9 @@ module Train::Platforms::Detect::Helpers
         @platform[:release] = payload["Version"]
         @platform[:name] = payload["Caption"]
 
-        read_wmic
+        # Prefer retrieving the OS details via `wmic` if available on the system to retain existing behavior.
+        # If `wmic` is not available, fall back to using CIM as an alternative method.
+        wmic_available? ? read_wmic : read_cim_os
         true
       rescue
         false
@@ -108,7 +113,7 @@ module Train::Platforms::Detect::Helpers
     def windows_uuid
       uuid = windows_uuid_from_chef
       uuid = windows_uuid_from_machine_file if uuid.nil?
-      uuid = windows_uuid_from_wmic if uuid.nil?
+      uuid = windows_uuid_from_wmic_or_cim if uuid.nil?
       uuid = windows_uuid_from_registry if uuid.nil?
       raise Train::TransportError, "Cannot find a UUID for your node." if uuid.nil?
 
@@ -134,11 +139,51 @@ module Train::Platforms::Detect::Helpers
       json["node_uuid"]
     end
 
+    def windows_uuid_from_wmic_or_cim
+      # Retrieve the Windows UUID using `wmic` if it is available and not marked as deprecated, maintaining compatibility with older systems.
+      # If `wmic` is unavailable or deprecated, use the `Get-CimInstance` command, which is the modern and recommended approach by Microsoft.
+      wmic_available? ? windows_uuid_from_wmic : windows_uuid_from_cim
+    end
+
     def windows_uuid_from_wmic
-      result = @backend.run_command("wmic csproduct get UUID")
+      # Switched from `wmic csproduct get UUID` to `wmic csproduct get UUID /value`
+      # to make the parsing of the UUID more reliable and consistent.
+      #
+      # When using the original `wmic csproduct get UUID` command, the output includes
+      # a header line and spacing that can vary depending on the system, making it harder
+      # to reliably extract the UUID. In some cases, splitting by line and taking the last
+      # element returns an empty string, even when exit_status is 0.
+      #
+      # Example:
+      #
+      # (byebug) result = @backend.run_command("wmic csproduct get UUID")
+      # #<struct Train::Extras::CommandResult stdout="UUID                                  \r\r\nEC20EBD7-8E03-06A8-645F-2D22E5A3BA4B  \r\r\n\r\r\n", stderr="", exit_status=0>
+      # (byebug) result.stdout
+      # "UUID                                  \r\r\nEC20EBD7-8E03-06A8-645F-2D22E5A3BA4B  \r\r\n\r\r\n"
+      # (byebug) result.exit_status
+      # 0
+      # (byebug) result.stdout.split("\r\n")[-1].strip
+      # ""
+      #
+      # In contrast, `wmic csproduct get UUID /value` returns a consistent `UUID=<value>` format,
+      # which is more suitable for regex matching.
+      #
+      # Example:
+      #
+      # byebug) result = @backend.run_command("wmic csproduct get UUID /value")
+      # #<struct Train::Extras::CommandResult stdout="\r\r\n\r\r\nUUID=EC20EBD7-8E03-06A8-645F-2D22E5A3BA4B\r\r\n\r\r\n\r\r\n\r\r\n", stderr="", exit_status=0>
+      # (byebug) result.stdout
+      # "\r\r\n\r\r\nUUID=EC20EBD7-8E03-06A8-645F-2D22E5A3BA4B\r\r\n\r\r\n\r\r\n\r\r\n"
+      # (byebug) result.stdout&.match(/UUID=([A-F0-9\-]+)/i)&.captures&.first
+      # "EC20EBD7-8E03-06A8-645F-2D22E5A3BA4B"
+      #
+      # This change improves parsing reliability and handles edge cases where the previous
+      # approach would return `nil` or raise errors on empty output lines.
+
+      result = @backend.run_command("wmic csproduct get UUID /value")
       return unless result.exit_status == 0
 
-      result.stdout.split("\r\n")[-1].strip
+      result.stdout&.match(/UUID=([A-F0-9\-]+)/i)&.captures&.first
     end
 
     def windows_uuid_from_registry
@@ -148,5 +193,63 @@ module Train::Platforms::Detect::Helpers
 
       result.stdout.chomp
     end
+
+    # Checks if `wmic` is available and not deprecated
+    def wmic_available?
+      # Return memoized value if already checked
+      return @wmic_available unless @wmic_available.nil?
+
+      # Runs the `wmic /?`` command, which provides help information for the WMIC (Windows Management Instrumentation Command-line) tool.
+      # It displays a list of available global switches and aliases, as well as details about their usage.
+      # The output also includes information about deprecated status for the 'wmic' tool.
+      result = @backend.run_command("wmic /?")
+
+      # Check if command ran successfully and output does not contain 'wmic is deprecated'
+      @wmic_available = result.exit_status == 0 && !(result.stdout.downcase.include?("wmic is deprecated"))
+    end
+
+    def read_cim_os
+      cmd = 'powershell -Command "Get-CimInstance Win32_OperatingSystem | Select-Object Caption, Version, BuildNumber | ConvertTo-Json"'
+      res = @backend.run_command(cmd)
+      return unless res.exit_status == 0
+
+      begin
+        sys_info = JSON.parse(res.stdout)
+        @platform[:release] = sys_info["Version"]
+        @platform[:build] = sys_info["BuildNumber"]
+        @platform[:name] = sys_info["Caption"]
+        @platform[:name] = @platform[:name].gsub("Microsoft", "").strip unless @platform[:name].empty?
+        @platform[:arch] = read_cim_cpu
+      rescue
+        nil
+      end
+    end
+
+    def read_cim_cpu
+      cmd = 'powershell -Command "(Get-CimInstance Win32_Processor).Architecture"'
+      res = @backend.run_command(cmd)
+      return unless res.exit_status == 0
+
+      arch_map = {
+        0 => "i386",
+        1 => "mips",
+        2 => "alpha",
+        3 => "powerpc",
+        5 => "arm",
+        6 => "ia64",
+        9 => "x86_64",
+      }
+
+      arch_map[res.stdout.strip.to_i]
+    end
+
+    def windows_uuid_from_cim
+      cmd = 'powershell -Command "(Get-CimInstance -Class Win32_ComputerSystemProduct).UUID"'
+      res = @backend.run_command(cmd)
+      return unless res.exit_status == 0
+
+      res.stdout.strip
+    end
+
   end
 end

data/lib/train/version.rb

@@ -2,5 +2,5 @@
 # Author:: Dominik Richter (<dominik.richter@gmail.com>)
 
 module Train
-  VERSION = "3.12.7".freeze
+  VERSION = "3.13.2".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: train-core
 version: !ruby/object:Gem::Version
-  version: 3.12.7
+  version: 3.13.2
 platform: ruby
 authors:
 - Chef InSpec Team
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-08-21 00:00:00.000000000 Z
+date: 2025-06-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -28,16 +28,16 @@ dependencies:
   name: ffi
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "!="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.13.0
+        version: 1.16.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "!="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.13.0
+        version: 1.16.0
 - !ruby/object:Gem::Dependency
   name: json
   requirement: !ruby/object:Gem::Requirement
@@ -183,14 +183,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.7'
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.3
+rubygems_version: 3.3.27
 signing_key: 
 specification_version: 4
 summary: Transport interface to talk to a selected set of backends.