Change in vdsm[ovirt-3.5]: jsonrpc: filter params while logging secure method invocation

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Sat May 16 20:56:25 UTC 2015


Piotr Kliczewski has posted comments on this change.

Change subject: jsonrpc: filter params while logging secure method invocation
......................................................................


Patch Set 2:

(3 comments)

https://gerrit.ovirt.org/#/c/40961/2/lib/yajsonrpc/__init__.py
File lib/yajsonrpc/__init__.py:

Line 494
Line 495
Line 496
Line 497
Line 498
> Same:
Done


Line 454:                    'StoragePool.connectStorageServer',
Line 455:                    'StoragePool.disconnectStorageServer',
Line 456:                    'StoragePool.validateStorageServerConnection',
Line 457:                    'VM.desktopLogin',
Line 458:                    'VM.setTicket')
> Host_fenceNode is missing - logging it only in TRACE level is not good enou
Done
Line 459: 
Line 460:     SECURE_RETURN = ('Host.getDeviceList')
Line 461: 
Line 462:     def __init__(self, bridge, threadFactory=None):


Line 474:                              'Host_getStats', 'StorageDomain_getStats',
Line 475:                              'VM_getStats', 'Host_fenceNode'):
Line 476:             logLevel = logging.TRACE
Line 477:         self._log_message(logLevel, 'Calling', self.SECURE_ARGS, req.method,
Line 478:                           req.params)
> This works, but is more complicated than needed, and is harder to follow. I
Done
Line 479: 
Line 480:         try:
Line 481:             method = getattr(self._bridge, mangledMethod)
Line 482:         except AttributeError:


-- 
To view, visit https://gerrit.ovirt.org/40961
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I34a36a361d2be28343352ae83ceac94b86162a12
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list