Fix arguments and docs (#2147)

- oauthkey shouldn't be logged
- action should be restricted and checked and the
doc should correspond to the code
This commit is contained in:
Michael Scherer 2016-05-02 19:21:02 +02:00 committed by Brian Coca
parent 12f1c85aa3
commit 7fd4180857

View file

@ -57,7 +57,7 @@ options:
description: description:
- This tells the githooks module what you want it to do. - This tells the githooks module what you want it to do.
required: true required: true
choices: [ "create", "cleanall" ] choices: [ "create", "cleanall", "list", "clean504" ]
validate_certs: validate_certs:
description: description:
- If C(no), SSL certificates for the target repo will not be validated. This should only be used - 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(): def main():
module = AnsibleModule( module = AnsibleModule(
argument_spec=dict( argument_spec=dict(
action=dict(required=True), action=dict(required=True, choices=['list','clean504','cleanall','create']),
hookurl=dict(required=False), hookurl=dict(required=False),
oauthkey=dict(required=True), oauthkey=dict(required=True, no_log=True),
repo=dict(required=True), repo=dict(required=True),
user=dict(required=True), user=dict(required=True),
validate_certs=dict(default='yes', type='bool'), validate_certs=dict(default='yes', type='bool'),