Fix #168 Default owner of 'root' fails on Windows

Signed-off-by: Christopher Williams <[email protected]>

libraries/default.rb

@@ -49,6 +49,7 @@ def set_paths
       new_resource.prefix_root = defaults.prefix_root
       new_resource.home_dir = defaults.home_dir
       new_resource.version = defaults.version
+      new_resource.owner = defaults.owner
 
       # TODO: what happens when the path is already set --
       #   with the current logic we overwrite it

libraries/resource_defaults.rb

@@ -33,6 +33,10 @@ def path
       end
     end
 
+    def owner
+      resource.owner || default_owner
+    end
+
     def windows?
       node_in_run_context['platform_family'] == 'windows'
     end
@@ -89,6 +93,21 @@ def default_version
       '1'
     end
 
+    def default_owner
+      if windows?
+        wmi_property_from_query(:name, "select * from Win32_UserAccount where sid like 'S-1-5-21-%-500' and LocalAccount=True")
+      else
+        'root'
+      end
+    end
+
+    def wmi_property_from_query(wmi_property, wmi_query)
+      @wmi = ::WIN32OLE.connect('winmgmts://')
+      result = @wmi.ExecQuery(wmi_query)
+      return nil unless result.each.count > 0
+      result.each.next.send(wmi_property)
+    end
+
     def file_cache_path
       Chef::Config[:file_cache_path]
     end

resources/default.rb

@@ -41,7 +41,7 @@
   :release_file,
   :version
 
-attribute :owner, kind_of: String, default: 'root'
+attribute :owner, kind_of: String, default: nil
 attribute :group, kind_of: [String, Fixnum], default: 0
 attribute :url, kind_of: String, required: true
 attribute :path, kind_of: String, default: nil