tasks/main.yml · c30a2214a00f10a66e71ae81f0c61a1ae4bd39fb · Lydra / YunoHost / ansible-yunohost

Blame

3 years ago

c2317717

refactor: Command ansible module across the repo for better security · c2317717

Arthur BOUDREAULT authored 3 years ago and

Christophe Chaudier committed 3 years ago

Shell module is not needed here.

According to various sources (https://www.youtube.com/watch?v=57gAqKvAKck or https://stackoverflow.com/questions/56663332/difference-between-shell-and-command-in-ansible) it is not useful to use shell ansible module when not working with operands. Therefore I have decided to switch every actions to command module, more secure. Ansible-lint says "Shell should only be used when piping, redirecting or chaining commands"

c2317717

History

refactor: Command ansible module across the repo for better security

Arthur BOUDREAULT authored 3 years ago and

Christophe Chaudier committed 3 years ago

Shell module is not needed here.

According to various sources (https://www.youtube.com/watch?v=57gAqKvAKck or https://stackoverflow.com/questions/56663332/difference-between-shell-and-command-in-ansible) it is not useful to use shell ansible module when not working with operands. Therefore I have decided to switch every actions to command module, more secure. Ansible-lint says "Shell should only be used when piping, redirecting or chaining commands"