x/vmware-nsx
Code Issues Proposed changes

remove token from notifier middleware

Browse Source 
notifier middleware is capturing token and sending it to MQ. this
is not advisable so we should filter it out.

Closes-Bug: #1321080
Change-Id: Ia1bfa1bd24989681db1d2f385defc12e69a01f8d
This commit is contained in:
Gordon Chung
2014-05-22 10:51:25 -04:00
parent c86ca6d8a7
commit cae39f8380
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
neutron/openstack/common/middleware/notifier.py
View File

@@ -24,7 +24,7 @@ import six
import webob.dec import webob.dec
from neutron.openstack.common import context from neutron.openstack.common import context
from neutron.openstack.common.gettextutils import _ from neutron.openstack.common.gettextutils import _LE
from neutron.openstack.common import log as logging from neutron.openstack.common import log as logging
from neutron.openstack.common.middleware import base from neutron.openstack.common.middleware import base
from neutron.openstack.common.notifier import api from neutron.openstack.common.notifier import api
@@ -37,8 +37,8 @@ def log_and_ignore_error(fn):
try: try:
return fn(*args, **kwargs) return fn(*args, **kwargs)
except Exception as e: except Exception as e:
LOG.exception(_('An exception occurred processing ' LOG.exception(_LE('An exception occurred processing '
'the API call: %s ') % e) 'the API call: %s ') % e)
return wrapped return wrapped
@@ -56,7 +56,7 @@ class RequestNotifier(base.Middleware):
return _factory return _factory
def __init__(self, app, **conf): def __init__(self, app, **conf):
self.service_name = conf.get('service_name', None) self.service_name = conf.get('service_name')
self.ignore_req_list = [x.upper().strip() for x in self.ignore_req_list = [x.upper().strip() for x in
conf.get('ignore_req_list', '').split(',')] conf.get('ignore_req_list', '').split(',')]
super(RequestNotifier, self).__init__(app) super(RequestNotifier, self).__init__(app)
@@ -68,7 +68,7 @@ class RequestNotifier(base.Middleware):
""" """
return dict((k, v) for k, v in six.iteritems(environ) return dict((k, v) for k, v in six.iteritems(environ)
if k.isupper()) if k.isupper() and k != 'HTTP_X_AUTH_TOKEN')
@log_and_ignore_error @log_and_ignore_error
def process_request(self, request): def process_request(self, request):