mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
Minor rework of pamd module. Fixed some documentation. (#19758)
* Minor rework. Fixed some documentation. * Requested changes in the case the arg is unicode
This commit is contained in:
parent
5536ffe978
commit
99e19ad617
1 changed files with 20 additions and 13 deletions
|
@ -17,6 +17,7 @@
|
|||
# You should have received a copy of the GNU General Public License
|
||||
# along with Ansible. If not, see <http://www.gnu.org/licenses/>.
|
||||
from ansible.module_utils.basic import AnsibleModule
|
||||
from ansible.module_utils.six import string_types
|
||||
from ansible.module_utils.pycompat24 import get_exception
|
||||
|
||||
DOCUMENTATION = """
|
||||
|
@ -74,9 +75,9 @@ options:
|
|||
missing from the existing rule. Furthermore, if the module argument
|
||||
takes a value denoted by '=', the value will be changed to that specified
|
||||
in module_arguments.
|
||||
insert:
|
||||
state:
|
||||
required: false
|
||||
default: none
|
||||
default: updated
|
||||
choices:
|
||||
- updated
|
||||
- before
|
||||
|
@ -133,7 +134,8 @@ EXAMPLES = """
|
|||
type: auth
|
||||
control: required
|
||||
module_path: pam_faillock.so
|
||||
new_type: auth new_control=sufficient
|
||||
new_type: auth
|
||||
new_control=sufficient
|
||||
new_module_path: pam_faillock.so
|
||||
state: after
|
||||
|
||||
|
@ -263,7 +265,6 @@ class PamdService(object):
|
|||
self.fname = self.path + "/" + self.name
|
||||
self.preamble = []
|
||||
self.rules = []
|
||||
|
||||
try:
|
||||
for line in open(self.fname, 'r'):
|
||||
if line.startswith('#') and not line.isspace():
|
||||
|
@ -380,11 +381,14 @@ def remove_module_arguments(service, old_rule, module_args):
|
|||
result = {'action': 'args_absent'}
|
||||
changed = False
|
||||
change_count = 0
|
||||
if isinstance(module_args, ansible.module_utils.six.string_types):
|
||||
module_args = module_args.split(' ')
|
||||
|
||||
for rule in service.rules:
|
||||
if (old_rule.rule_type == rule.rule_type and
|
||||
old_rule.rule_control == rule.rule_control and
|
||||
old_rule.rule_module_path == rule.rule_module_path):
|
||||
for arg_to_remove in module_args.split():
|
||||
for arg_to_remove in module_args:
|
||||
for arg in rule.rule_module_args:
|
||||
if arg == arg_to_remove:
|
||||
rule.rule_module_args.remove(arg)
|
||||
|
@ -401,11 +405,14 @@ def add_module_arguments(service, old_rule, module_args):
|
|||
result = {'action': 'args_present'}
|
||||
changed = False
|
||||
change_count = 0
|
||||
if isinstance(module_args, ansible.module_utils.six.string_types):
|
||||
module_args = module_args.split(' ')
|
||||
|
||||
for rule in service.rules:
|
||||
if (old_rule.rule_type == rule.rule_type and
|
||||
old_rule.rule_control == rule.rule_control and
|
||||
old_rule.rule_module_path == rule.rule_module_path):
|
||||
for arg_to_add in module_args.split(' '):
|
||||
for arg_to_add in module_args:
|
||||
if "=" in arg_to_add:
|
||||
pre_string = arg_to_add[:arg_to_add.index('=')+1]
|
||||
indicies = [i for i, arg
|
||||
|
@ -450,22 +457,22 @@ def main():
|
|||
|
||||
module = AnsibleModule(
|
||||
argument_spec=dict(
|
||||
name=dict(required=True),
|
||||
name=dict(required=True, type='str'),
|
||||
type=dict(required=True,
|
||||
choices=['account', 'auth',
|
||||
'password', 'session']),
|
||||
control=dict(required=True),
|
||||
module_path=dict(required=True),
|
||||
control=dict(required=True, type='str'),
|
||||
module_path=dict(required=True, type='str'),
|
||||
new_type=dict(required=False,
|
||||
choices=['account', 'auth',
|
||||
'password', 'session']),
|
||||
new_control=dict(required=False),
|
||||
new_module_path=dict(required=False),
|
||||
module_arguments=dict(required=False),
|
||||
new_control=dict(required=False, type='str'),
|
||||
new_module_path=dict(required=False, type='str'),
|
||||
module_arguments=dict(required=False, type='list'),
|
||||
state=dict(required=False, default="updated",
|
||||
choices=['before', 'after', 'updated',
|
||||
'args_absent', 'args_present']),
|
||||
path=dict(required=False, default='/etc/pam.d')
|
||||
path=dict(required=False, default='/etc/pam.d', type='str')
|
||||
),
|
||||
supports_check_mode=True
|
||||
)
|
||||
|
|
Loading…
Reference in a new issue