chef-powershell 2.18.5 → 3.0.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 36453d0835276a228072f5d76630d75a6d3ecc12a5902b0a973427cc12790fe4
-  data.tar.gz: 19c1a4cb862cda9518f107714dcf6b8f90198339ffb30450391d1aaaa977bcf5
+  metadata.gz: 4d93049c98f0e42d1cf2760cf353eb4fb16d0259ff0fec3790e0d1763a835bc4
+  data.tar.gz: 2a162dc437e62342bf1fd54fdbf5d0aa82b77047320135c27c40b1672e6d97b5
 SHA512:
-  metadata.gz: 342c398f8a5ec0259b6c4552708067cf3ecaa38c4f5fbdd0224ca0aa7ac0956b9e9c5740d854e39d7218173cd04d5202f3fc590fc64a2a2a9bf9adf3ad133419
-  data.tar.gz: 5c8ca7ceab568a1f95a2233be781ea7367629fd1ac0c98a257bbe18f664e7e3c2eb044a0d3da5078800b1f2a37afa5a24f3f62146ff12a2316af750a26655dc7
+  metadata.gz: e13dad4b1a55c8af3bcdffdc0728ccd4bea6eeaa1d75cd17d3d3f1cffb2bb5393c42fa3f8138c2c4258f54d7ef7d66187d6a5433d77f46042db880b2577f4ce4
+  data.tar.gz: 4008e7bf79b9df4de1cc06facb61a9938cdddff1903880df37b3af3ba81cf01e2a99d00d13e11e1e1f84e49231bd2bab34d3dc8c2947ae81f43592c678499268

data/chef-powershell.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.required_ruby_version = ">= 2.6"
 
   spec.add_runtime_dependency "ffi", "~> 1.15"
-  spec.add_runtime_dependency "ffi-yajl", "~> 2.4"
+  spec.add_runtime_dependency "ffi-yajl", "~> 3.0"
 
   spec.metadata = {
     "bug_tracker_uri" => "https://github.com/chef/chef/issues",

data/lib/chef-powershell/powershell.rb

@@ -64,10 +64,6 @@ class ChefPowerShell
     end
 
     module PowerMod
-      class << self
-        attr_accessor :result_string, :retry_count, :exception, :result, :errors, :verbose, :hashed_outcome
-      end
-
       extend FFI::Library
       # FFI requires attaching to modules, not classes, so we need to
       # have a module here. The module level variables *could* be refactored
@@ -76,26 +72,17 @@ class ChefPowerShell
       @@ps_command = ""
       @@ps_timeout = -1
 
-      StoreResultCallback = FFI::Function.new(:bool, [:pointer, :size_t]) do |data, size|
-        # try parsing the result *first* before returning from the function, and if it fails,
-        # return false so that the function can be retried from the C++ side.
-        begin
-          #          @@result_string = data.get_bytes(0, size).force_encoding("UTF-16LE").encode("UTF-8")
-          @result_string = data.get_bytes(0, size).force_encoding("UTF-16LE").encode("UTF-8").bytes.reverse.drop_while {|x| x == 0}.reverse.pack("C*")
-          @hashed_outcome = FFI_Yajl::Parser.parse(@result_string)
-          @result = FFI_Yajl::Parser.parse(@hashed_outcome["result"])
-          @errors = @hashed_outcome["errors"]
-          @verbose = @hashed_outcome["verbose"]
-          true
-        rescue => e
-          @retry_count += 1
-          puts "#{e.inspect}"
-          @exception = e
-          return true if @retry_count > 3
-          puts "Retrying PowerShell command execution #{@retry_count}"
-          sleep 1
-          false
-        end
+      AllocateCallback = FFI::Function.new(:pointer, [:size_t]) do |size|
+        # Capture the pointer here so that Ruby knows that it is an
+        # FFI::MemoryPointer that can receive &.free. If you try to
+        # free the pointer from execute_powershell, Ruby will not have
+        # access to the type information and will core dump spectacularly.
+        @@pointer = FFI::MemoryPointer.new(:uchar, size)
+      end
+
+      def self.free_pointer
+        @@pointer&.free
+        @@pointer = nil
       end
 
       def self.set_ps_dll(value)
@@ -111,12 +98,10 @@ class ChefPowerShell
       end
 
       def self.do_work
-        @exception = nil
-        @retry_count = 0
         ffi_lib @@powershell_dll
         attach_function :execute_powershell, :ExecuteScript, %i{string int pointer}, :pointer
 
-        execute_powershell(@@ps_command, @@ps_timeout, StoreResultCallback)
+        execute_powershell(@@ps_command, @@ps_timeout, AllocateCallback)
       end
     end
 
@@ -132,10 +117,13 @@ class ChefPowerShell
 
       PowerMod.set_ps_command(script)
       execution = PowerMod.do_work
-      raise PowerMod.exception if PowerMod.exception
-      @result = PowerMod.result
-      @errors = PowerMod.errors
-      @verbose = PowerMod.verbose
+      output = execution.read_utf16string
+      hashed_outcome = FFI_Yajl::Parser.parse(output)
+      @result = FFI_Yajl::Parser.parse(hashed_outcome["result"])
+      @errors = hashed_outcome["errors"]
+      @verbose = hashed_outcome["verbose"]
+    ensure
+      PowerMod.free_pointer
     end
   end
 end

data/lib/chef-powershell/version.rb

@@ -16,5 +16,5 @@
 
 module ChefPowerShellModule
   CHEFPOWERSHELL_ROOT = File.expand_path("..", __dir__)
-  VERSION = "2.18.5".freeze
+  VERSION = "3.0.30".freeze
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: chef-powershell
 version: !ruby/object:Gem::Version
-  version: 2.18.5
+  version: 3.0.30
 platform: ruby
 authors:
 - Chef Software, Inc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-07-11 00:00:00.000000000 Z
+date: 2023-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ffi
@@ -30,14 +30,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '3.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '2.4'
+        version: '3.0'
 description: 
 email:
 - oss@chef.io