summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Smith <tsmith84@gmail.com>2020-07-06 20:26:29 -0700
committerTim Smith <tsmith84@gmail.com>2020-07-20 11:45:27 -0700
commit26e9917e4d99ff3aaf4a78138016af46d5b5f9e9 (patch)
tree1e93df65e8612ff2a71adf63444bd3df9bbe2d65
parent317dc9c3e2f642bf4e3777fe0745137af9f54413 (diff)
downloadchef-26e9917e4d99ff3aaf4a78138016af46d5b5f9e9.tar.gz
Add logging and fix shellout with user property
Signed-off-by: Tim Smith <tsmith@chef.io>
-rw-r--r--lib/chef/resource/macos_userdefaults.rb11
1 files changed, 7 insertions, 4 deletions
diff --git a/lib/chef/resource/macos_userdefaults.rb b/lib/chef/resource/macos_userdefaults.rb
index e2865b16af..571786fad0 100644
--- a/lib/chef/resource/macos_userdefaults.rb
+++ b/lib/chef/resource/macos_userdefaults.rb
@@ -16,7 +16,7 @@
#
require_relative "../resource"
-require 'shellwords'
+require "shellwords"
class Chef
class Resource
@@ -109,15 +109,18 @@ class Chef
load_current_value do |desired|
coerced_value = coerce_booleans(desired.value)
- state_cmd = ['/usr/bin/defaults', 'read', desired.domain, desired.key]
+ state_cmd = ["/usr/bin/defaults", "read", desired.domain, desired.key]
state = if desired.user.nil?
shell_out(state_cmd)
else
- shell_out(cmd, user: desired.user)
+ shell_out(state_cmd, user: desired.user)
end
- current_value_does_not_exist! if state.error?
+ if state.error?
+ Chef::Log.debug "#load_current_value: #{state_cmd.join(" ")} returned stdout: #{state.stdout} and stderr: #{state.stderr}"
+ current_value_does_not_exist!
+ end
value state.stdout.strip
end