Yaniv Bronhaim has submitted this change and it was merged. Change subject: Revert "jsonrpc: Accept all stomp commands" ......................................................................
Revert "jsonrpc: Accept all stomp commands" There is no verification whether incoming stomp messages are correct for correct runtime context. This work is planned for next release. We can use protocol detection as sanity check so we will recieve only messages which we know are correct. This reverts commit 6c2892a2a0bc337dcd4a40da54d08d3d63f95028. Change-Id: I3aef9781e09cb582e9a5422638b82327974f1213 Signed-off-by: pkliczewski <[email protected]> Reviewed-on: http://gerrit.ovirt.org/30033 Reviewed-by: Yaniv Bronhaim <[email protected]> --- M lib/yajsonrpc/stomp.py M lib/yajsonrpc/stompReactor.py 2 files changed, 2 insertions(+), 6 deletions(-) Approvals: Piotr Kliczewski: Verified Yaniv Bronhaim: Looks good to me, approved Objections: Nir Soffer: I would prefer that you didn't submit this -- To view, visit http://gerrit.ovirt.org/30033 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3aef9781e09cb582e9a5422638b82327974f1213 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
