Commit 81953754 by Stephen Fromm

Sanitize possible password argument when logging invocation; taken from user module

parent baf07659
...@@ -33,6 +33,7 @@ try: ...@@ -33,6 +33,7 @@ try:
except ImportError: except ImportError:
import simplejson as json import simplejson as json
import os import os
import re
import shlex import shlex
import subprocess import subprocess
import sys import sys
...@@ -118,7 +119,9 @@ class AnsibleModule(object): ...@@ -118,7 +119,9 @@ class AnsibleModule(object):
def _log_invocation(self): def _log_invocation(self):
''' log that ansible ran the module ''' ''' log that ansible ran the module '''
syslog.openlog('ansible-%s' % os.path.basename(__file__)) syslog.openlog('ansible-%s' % os.path.basename(__file__))
syslog.syslog(syslog.LOG_NOTICE, 'Invoked with %s' % self.args) # Sanitize possible password argument when logging
log_args = re.sub(r'password=.+ (.*)', r"password=NOT_LOGGING_PASSWORD \1", self.args)
syslog.syslog(syslog.LOG_NOTICE, 'Invoked with %s' % log_args)
def exit_json(self, **kwargs): def exit_json(self, **kwargs):
''' return from the module, without error ''' ''' return from the module, without error '''
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment