Fix #1809, use the proper method to fail

9 years ago · 6b2bc9df04
parent db32172959
commit 6b2bc9df04
1 changed files with 1 additions and 1 deletions
--- a/system/firewalld.py
+++ b/system/firewalld.py
@ -260,7 +260,7 @@ def main():
        supports_check_mode=True
    )
    if module.params['source'] == None and module.params['permanent'] == None:
-        module.fail(msg='permanent is a required parameter')
+        module.fail_json(msg='permanent is a required parameter')

    if not HAS_FIREWALLD:
        module.fail_json(msg='firewalld and its python 2 module are required for this module')