Browse Source

remove token from notifier middleware

oslo-incubator sync to address the security bug
in middleware (as below).

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

Change-Id: Ia1bfa1bd24989681db1d2f385defc12e69a01f8d
Closes-Bug: #1321080
changes/99/101799/1
Grant Murphy 8 years ago
parent
commit
264f3b0d96
  1. 2
      ceilometer/openstack/common/middleware/audit.py
  2. 2
      ceilometer/openstack/common/middleware/notifier.py

2
ceilometer/openstack/common/middleware/audit.py

@ -1,6 +1,6 @@
# vim: tabstop=4 shiftwidth=4 softtabstop=4
# Copyright (c) 2013 OpenStack LLC.
# Copyright (c) 2013 OpenStack Foundation
# All Rights Reserved.
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may

2
ceilometer/openstack/common/middleware/notifier.py

@ -66,7 +66,7 @@ class RequestNotifier(base.Middleware):
"""
return dict((k, v) for k, v in environ.iteritems()
if k.isupper())
if k.isupper() and k != 'HTTP_X_AUTH_TOKEN')
@log_and_ignore_error
def process_request(self, request):

Loading…
Cancel
Save