plugin: The example plugin can now return errors

Signed-off-by: Christian Decker <decker.christian@gmail.com>

contrib/plugins/helloworld.py

@@ -19,6 +19,10 @@ def json_hello(request, name):
     return greeting
 
 
+def json_fail(request):
+    raise ValueError("This will fail")
+
+
 def json_getmanifest(request):
     global greeting
     return {
@@ -33,6 +37,10 @@ def json_getmanifest(request):
                 "name": "hello",
                 "description": "Returns a personalized greeting for {name}",
             },
+            {
+                "name": "fail",
+                "description": "Always returns a failure for testing",
+            },
         ]
     }
 
@@ -48,6 +56,7 @@ def json_init(request, options):
 
 methods = {
     'hello': json_hello,
+    'fail': json_fail,
     'getmanifest': json_getmanifest,
     'init': json_init,
 }
@@ -55,25 +64,33 @@ methods = {
 
 partial = ""
 for l in sys.stdin:
-    partial += l
     try:
+        partial += l
         request = json.loads(partial)
-        result = None
+    except Exception:
-        method = methods[request['method']]
+        continue
-        params = request['params']
+
+    result = None
+    method = methods[request['method']]
+    params = request['params']
+    try:
         if isinstance(params, dict):
             result = method(request, **params)
         else:
             result = method(request, *params)
-
         result = {
             "jsonrpc": "2.0",
             "result": result,
             "id": request['id']
         }
-        json.dump(result, fp=sys.stdout)
+    except Exception as e:
-        sys.stdout.write('\n')
+        result = {
-        sys.stdout.flush()
+            "jsonrpc": "2.0",
-        partial = ""
+            "error": "Error while processing {}".format(request['method']),
-    except Exception:
+            "id": request['id']
-        pass
+        }
+
+    json.dump(result, fp=sys.stdout)
+    sys.stdout.write('\n')
+    sys.stdout.flush()
+    partial = ""