puppet-cron icon indicating copy to clipboard operation
puppet-cron copied to clipboard

Changing file permissions to be uniform across all frequencies

Open jrk07 opened this issue 8 years ago • 3 comments

Currently the permissions for daily, hourly, and monthly are all 0644, while weekly is 0640. Recommending it is updated to 0644 to be uniform across the board unless there is a particular reason weekly should be different.


This change is Reviewable

jrk07 avatar Aug 31 '16 23:08 jrk07

Please note that the checks failing has nothing to do with this PR itself. Recommended this be pulled into master so that it does not impact future consumers of this module.

jrk07 avatar Sep 25 '16 03:09 jrk07

+1

davidnewhall avatar Dec 06 '16 09:12 davidnewhall

@jrk07 Just curious, what's your rationale for 644 vs. 640? 640 seems to be a more secure alternative, while cron, running as root, would still be able to read it.

TomyLobo avatar Jan 17 '18 09:01 TomyLobo