operation_log: Honor settings parameters

Previously 'format' and 'mask_fields' in OPERATION_LOG_OPTIONS
were completely ignored. OPERATION_LOG_OPTIONS is a dict, but
the logic tried to get attributes, so these options were ignored.

Change-Id: Ide4826ebf74218873a81b67aaa0dd817c4798702
Closes-Bug: #1675176
(cherry picked from commit fdb21222b6)
This commit is contained in:
Akihiro Motoki 2017-03-22 21:44:27 +00:00
parent 9dda5ad25e
commit bedd32243a
1 changed files with 2 additions and 2 deletions

View File

@ -56,8 +56,8 @@ class OperationLogMiddleware(object):
" [%(request_scheme)s] [%(referer_url)s] [%(request_url)s]"
" [%(message)s] [%(method)s] [%(http_status)s] [%(param)s]")
self.target_methods = [x for x in _methods if x in _available_methods]
self.mask_fields = getattr(_log_option, "mask_fields", ['password'])
self.format = getattr(_log_option, "format", _default_format)
self.mask_fields = _log_option.get("mask_fields", ['password'])
self.format = _log_option.get("format", _default_format)
self.static_rule = ['/js/', '/static/']
self._logger = logging.getLogger('horizon.operation_log')