The file permission of the cronjob definition is too strict

jgeusebroek / ansible-role-maxmind-geoip

An Ansible Role that installs and configures the Maxmind GeoIP databases

4 stars 9 forks source link

The file permission of the cronjob definition is too strict #9

Open brlin-tw opened 3 years ago

brlin-tw commented 3 years ago

https://github.com/jgeusebroek/ansible-role-maxmind-geoip/blob/b9e8798dc9cc4321e3dc5fcdb8fca9141076fa8f/tasks/configuration.yml#L11-L17

As the file doesn't contain any sensitive information it should be 644 instead of 640 so that a sudoer can easily check the content without elevating privileges.