diff vamp-server/convert.cpp @ 180:bd543e74a9bf

Correct the inspection of the JSON object in successful to look for both error and success objects, writing out an error string if neither are present. Revert error handling in readInput() for JSON.

--- a/vamp-server/convert.cpp	Fri Feb 03 11:09:37 2017 +0000
+++ b/vamp-server/convert.cpp	Fri Feb 03 11:12:27 2017 +0000
@@ -558,15 +558,8 @@
     if (format == "json") {
         string err;
         auto result = readInputJson(direction, err, eof);
-        const bool isRecognisedError = !result.success && result.errorText != "";
-        
-        // if the RequestOrResponse (result) has been populated with success=false and an error message
-        // then the server returned a well formed error, it is safe to return it for conversion 
-        // -- but if err is populated, something else has gone wrong
-        if (isRecognisedError || err == "") 
-            return result;
-        else 
-            throw runtime_error(err);
+        if (err != "") throw runtime_error(err);
+        else return result;
     } else if (format == "capnp") {
         return readInputCapnp(direction, eof);
     } else {