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:
parent
9dda5ad25e
commit
bedd32243a
|
@ -56,8 +56,8 @@ class OperationLogMiddleware(object):
|
||||||
" [%(request_scheme)s] [%(referer_url)s] [%(request_url)s]"
|
" [%(request_scheme)s] [%(referer_url)s] [%(request_url)s]"
|
||||||
" [%(message)s] [%(method)s] [%(http_status)s] [%(param)s]")
|
" [%(message)s] [%(method)s] [%(http_status)s] [%(param)s]")
|
||||||
self.target_methods = [x for x in _methods if x in _available_methods]
|
self.target_methods = [x for x in _methods if x in _available_methods]
|
||||||
self.mask_fields = getattr(_log_option, "mask_fields", ['password'])
|
self.mask_fields = _log_option.get("mask_fields", ['password'])
|
||||||
self.format = getattr(_log_option, "format", _default_format)
|
self.format = _log_option.get("format", _default_format)
|
||||||
self.static_rule = ['/js/', '/static/']
|
self.static_rule = ['/js/', '/static/']
|
||||||
self._logger = logging.getLogger('horizon.operation_log')
|
self._logger = logging.getLogger('horizon.operation_log')
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue