Piotr Kliczewski has uploaded a new change for review. Change subject: Method not found should return proper code ......................................................................
Method not found should return proper code When a method is not found we should return -32601 code. Change-Id: I9b3e1360303bca7ae3c30846ad46fec1fadac7fb Signed-off-by: Piotr Kliczewski <[email protected]> --- M lib/vdsm/rpc/Bridge.py 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/20/63120/1 diff --git a/lib/vdsm/rpc/Bridge.py b/lib/vdsm/rpc/Bridge.py index bd180e0..0bd1aaa 100644 --- a/lib/vdsm/rpc/Bridge.py +++ b/lib/vdsm/rpc/Bridge.py @@ -110,7 +110,7 @@ try: className, methodName = method.split('.', 1) self._schema.get_method(vdsmapi.MethodRep(className, methodName)) - except (KeyError, ValueError): + except (vdsmapi.MethodNotFound, ValueError): raise yajsonrpc.JsonRpcMethodNotFoundError(method) return partial(self._dynamicMethod, className, methodName) -- To view, visit https://gerrit.ovirt.org/63120 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I9b3e1360303bca7ae3c30846ad46fec1fadac7fb Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
