From 7fd4180857f856a59792724e02e95dd99c067083 Mon Sep 17 00:00:00 2001 From: Michael Scherer Date: Mon, 2 May 2016 19:21:02 +0200 Subject: [PATCH] Fix arguments and docs (#2147) - oauthkey shouldn't be logged - action should be restricted and checked and the doc should correspond to the code --- source_control/github_hooks.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/source_control/github_hooks.py b/source_control/github_hooks.py index 9f664875587..8d3c120a787 100644 --- a/source_control/github_hooks.py +++ b/source_control/github_hooks.py @@ -57,7 +57,7 @@ options: description: - This tells the githooks module what you want it to do. required: true - choices: [ "create", "cleanall" ] + choices: [ "create", "cleanall", "list", "clean504" ] validate_certs: description: - If C(no), SSL certificates for the target repo will not be validated. This should only be used @@ -152,9 +152,9 @@ def _delete(module, hookurl, oauthkey, repo, user, hookid): def main(): module = AnsibleModule( argument_spec=dict( - action=dict(required=True), + action=dict(required=True, choices=['list','clean504','cleanall','create']), hookurl=dict(required=False), - oauthkey=dict(required=True), + oauthkey=dict(required=True, no_log=True), repo=dict(required=True), user=dict(required=True), validate_certs=dict(default='yes', type='bool'),