Issue #22064 has been updated by Josh Cooper. Status changed from Merged - Pending Release to Accepted
This caused problems on some versions of Windows as the installer was creating an access control entry for an invalid SID (S-1-5-32). As a result, you couldn't run puppet interactively and the facts.d directory existed, such as during PE install. This has been reverted in commit [e3f6052](https://github.com/puppetlabs/puppet_for_the_win/commit/e3f60524e047de0f031f8ff4e2a9422c4c7499bc). ---------------------------------------- Bug #22064: Potential Local Escalation issue with Facts.d folder for executable facts on Windows https://projects.puppetlabs.com/issues/22064#change-96339 * Author: Rob Reynolds * Status: Accepted * Priority: High * Assignee: Rob Reynolds * Category: * Target version: 1.7.3 * Keywords: windows * Branch: https://github.com/puppetlabs/puppet_for_the_win/pull/49 * Affected Facter version: ---------------------------------------- When we enable executable facts, we need to ensure the facts.d folder is locked down by the installer. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at http://groups.google.com/group/puppet-bugs. For more options, visit https://groups.google.com/groups/opt_out.
