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
(cherry picked from commit fd2a66f9a3)
This commit is contained in:
ZhiQiang Fan 2015-03-17 17:29:11 +08:00 committed by gordon chung
parent 392054f13f
commit 8204521e99
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

@ -22,7 +22,6 @@
from oslo.config import cfg
import oslo.messaging
from ceilometer.openstack.common.gettextutils import _
from ceilometer.openstack.common import log
from ceilometer import plugin
from ceilometer import sample
@ -74,7 +73,6 @@ class NetworkNotificationBase(plugin.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)