search

robertdebock / ansible-role-cups

Install and configure CUPS, the Common Unix Printing System.
https://robertdebock.nl/
Apache License 2.0
8 stars 5 forks source link

Add the allow field to all the locations #2

Closed dfein38347g closed 2 years ago

dfein38347g commented 2 years ago

The template did not include the allow field to be inserted into the configuration, therefore not allowing remote users to access the CUPS web interface.

Fixes Issue #1

robertdebock commented 2 years ago

May I also ask you to describe an example in molecule/default/converge.yml? That's a good test and help users of the role understand how to use the allow option.

dfein38347g commented 2 years ago

May I also ask you to describe an example in molecule/default/converge.yml? That's a good test and help users of the role understand how to use the allow option.

I'm new to ansible and this is my first pull request. I'm not sure what you mean. But the allow field is already in your role variables on your readme, this is just fixing a bug in the template where the allow field wasn't used.