Fix no_log value for content in common file args settings

Also adds a test to ensure the content value is not logged.

Related to #8647
This commit is contained in:
James Cammarata 2014-08-19 10:46:46 -05:00
parent 8a5675ca9f
commit 3a7aca6066
2 changed files with 4 additions and 2 deletions

View file

@ -152,7 +152,7 @@ FILE_COMMON_ARGUMENTS=dict(
selevel = dict(),
setype = dict(),
# not taken by the file module, but other modules call file so it must ignore them.
content = dict(),
content = dict(no_log=True),
backup = dict(),
force = dict(),
remote_src = dict(), # used by assemble
@ -884,7 +884,7 @@ class AnsibleModule(object):
arg_opts = self.argument_spec.get(canon, {})
no_log = arg_opts.get('no_log', False)
if no_log:
if self.boolean(no_log):
log_args[param] = 'NOT_LOGGING_PARAMETER'
elif param in passwd_keys:
log_args[param] = 'NOT_LOGGING_PASSWORD'