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 <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
