wiggels / ansible-role-snipe-it

Ansible role to install, configure, and update a Snipe-IT server
https://galaxy.ansible.com/wiggels/snipeit
MIT License
5 stars 7 forks source link

Check for existence of snipe-it folder #1

Closed p-rintz closed 2 years ago

p-rintz commented 2 years ago

Previously, the role would fail if the snipe-it folder didnt exist previously. This gets now checked and the folder created if it doesnt already exist.

wiggels commented 2 years ago

Looks like a good addition, I would only request that you change the permissions to 755 before I merge it. 753 leaves the ability for everyone to write to that directory.

p-rintz commented 2 years ago

Quite correct. That was a stupid error. It’s been changed.