| Bill Sirinek, Jim Richardson, Nate McCurdy, Raul Tambre, Maximilian Philipps since you commented on PUP-5921, I wanted to collect your feedback on this ticket. I understand preserving file/directory modes is useful in some cases, especially for recursive directories, so I'm not against undeprecating it. But I have a harder time with uid/gid, especially when it's not 0. Assigning uid=502 across hosts is inviting a security problem. And then there are cross platforms concerns with Windows hosts. For this ticket what if we added a new value for the source_permissions property, something like mode, which only applies the mode from the source, but we keep use and use_when_creating deprecated, and in puppet 7 we drop use and use_when_creating? Julian Puppet is not really designed for the use case outlined in https://www.theurbanpenguin.com/puppet-moving-directories/. It may "work", but it's not going to scale for large numbers of files, it'll be slow, and you're better off using a bolt command for that. |
