From 40507220b4e3a65c92c2c1c68d994baa5b5a33d5 Mon Sep 17 00:00:00 2001 From: Nilashish Chakraborty Date: Wed, 3 Apr 2019 21:42:34 +0530 Subject: [PATCH] asa: Not all modules will have 'passwords' and 'context' keys (#54783) * Add a check of passwords Signed-off-by: NilashishC * Add caution for context key as well Signed-off-by: NilashishC * Update lib/ansible/module_utils/network/asa/asa.py Co-Authored-By: NilashishC * Update lib/ansible/module_utils/network/asa/asa.py Co-Authored-By: NilashishC --- lib/ansible/module_utils/network/asa/asa.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lib/ansible/module_utils/network/asa/asa.py b/lib/ansible/module_utils/network/asa/asa.py index d9b2ba2839..6d20a8c39a 100644 --- a/lib/ansible/module_utils/network/asa/asa.py +++ b/lib/ansible/module_utils/network/asa/asa.py @@ -86,7 +86,8 @@ def get_connection(module): return _CONNECTION _CONNECTION = Connection(module._socket_path) - context = module.params['context'] + # Not all modules include the 'context' key. + context = module.params.get('context') if context: if context == 'system': @@ -130,7 +131,8 @@ def run_commands(module, commands, check_rc=True): def get_config(module, flags=None): flags = [] if flags is None else flags - passwords = module.params['passwords'] + # Not all modules include the 'passwords' key. + passwords = module.params.get('passwords', False) if passwords: cmd = 'more system:running-config' else: