ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

Fix pamd module (#20048)

Browse source 
* Fix string_types usage

* Fix adding arguments for argument=value arguments

* Remove unnecesary check on module_args

* Fix argument requirements

* Proper handling of parameter=value arguments
This commit is contained in:
Valentín Gutiérrez 2017-01-09 18:30:32 +01:00 committed by Toshio Kuratomi
parent 457e94d21e
commit 6c448d3673
1 changed files with 21 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

36
lib/ansible/modules/system/pamd.py
View file

@ -17,7 +17,6 @@
# You should have received a copy of the GNU General Public License # You should have received a copy of the GNU General Public License
# along with Ansible. If not, see <http://www.gnu.org/licenses/>. # along with Ansible. If not, see <http://www.gnu.org/licenses/>.
from ansible.module_utils.basic import AnsibleModule from ansible.module_utils.basic import AnsibleModule
from ansible.module_utils.six import string_types
from ansible.module_utils.pycompat24 import get_exception from ansible.module_utils.pycompat24 import get_exception
DOCUMENTATION = """ DOCUMENTATION = """
@ -381,8 +380,6 @@ def remove_module_arguments(service, old_rule, module_args):
result = {'action': 'args_absent'} result = {'action': 'args_absent'}
changed = False changed = False
change_count = 0 change_count = 0
if isinstance(module_args, ansible.module_utils.six.string_types):
module_args = module_args.split(' ')
for rule in service.rules: for rule in service.rules:
if (old_rule.rule_type == rule.rule_type and if (old_rule.rule_type == rule.rule_type and
@ -405,8 +402,6 @@ def add_module_arguments(service, old_rule, module_args):
result = {'action': 'args_present'} result = {'action': 'args_present'}
changed = False changed = False
change_count = 0 change_count = 0
if isinstance(module_args, ansible.module_utils.six.string_types):
module_args = module_args.split(' ')
for rule in service.rules: for rule in service.rules:
if (old_rule.rule_type == rule.rule_type and if (old_rule.rule_type == rule.rule_type and
@ -418,15 +413,22 @@ def add_module_arguments(service, old_rule, module_args):
indicies = [i for i, arg indicies = [i for i, arg
in enumerate(rule.rule_module_args) in enumerate(rule.rule_module_args)
if arg.startswith(pre_string)] if arg.startswith(pre_string)]
for i in indicies: if len(indicies) == 0:
if rule.rule_module_args[i] != arg_to_add: rule.rule_module_args.append(arg_to_add)
rule.rule_module_args[i] = arg_to_add changed = True
changed = True result['added_arg_'+str(change_count)] = arg_to_add
result['updated_arg_' + result['to_rule_'+str(change_count)] = str(rule)
str(change_count)] = arg_to_add change_count += 1
result['in_rule_' + else:
str(change_count)] = str(rule) for i in indicies:
change_count += 1 if rule.rule_module_args[i] != arg_to_add:
rule.rule_module_args[i] = arg_to_add
changed = True
result['updated_arg_' +
str(change_count)] = arg_to_add
result['in_rule_' +
str(change_count)] = str(rule)
change_count += 1
elif arg_to_add not in rule.rule_module_args: elif arg_to_add not in rule.rule_module_args:
rule.rule_module_args.append(arg_to_add) rule.rule_module_args.append(arg_to_add)
changed = True changed = True
@ -474,7 +476,11 @@ def main():
'args_absent', 'args_present']), 'args_absent', 'args_present']),
path=dict(required=False, default='/etc/pam.d', type='str') path=dict(required=False, default='/etc/pam.d', type='str')
), ),
supports_check_mode=True supports_check_mode=True,
required_if=[
("state", "args_present", ["module_arguments"]),
("state", "args_absent", ["module_arguments"])
]
) )
service = module.params['name'] service = module.params['name']