openstack
/
ceilometer
Code Issues Proposed changes

remove log message when process notification 

message may contain some sensitive information, for example, auth
token. This patch removes log entire message in network.notications

Change-Id: I7e5f37668ab2a8bcf191ad886ac54352727272f7
Closes-Bug: #1433004
This commit is contained in:
ZhiQiang Fan 2015-03-17 17:29:11 +08:00
parent 3f9e48155a
commit fd2a66f9a3
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ceilometer/network/notifications.py
View File

@ -21,7 +21,6 @@ import oslo.messaging
from oslo_config import cfg
from ceilometer.agent import plugin_base
from ceilometer.i18n import _
from ceilometer.openstack.common import log
from ceilometer import sample
@ -72,7 +71,6 @@ class NetworkNotificationBase(plugin_base.NotificationBase):
for topic in conf.notification_topics]
def process_notification(self, message):
LOG.info(_('network notification %r') % message)
counter_name = getattr(self, 'counter_name', self.resource_name)
unit_value = getattr(self, 'unit', self.resource_name)