Merge pull request #203 from ryanbrandenburg/rybrande/NullResultFix

Always return result property on success
OmniSharp · Jan 21, 2020 · a8e44c2 · a8e44c2
2 parents 3bd6340 + 66b13ce
commit a8e44c2
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/JsonRpc/Serialization/Converters/ClientResponseConverter.cs b/src/JsonRpc/Serialization/Converters/ClientResponseConverter.cs
@@ -21,11 +21,14 @@ public override void WriteJson(JsonWriter writer, Response value, JsonSerializer
             writer.WriteValue("2.0");
             writer.WritePropertyName("id");
             writer.WriteValue(value.Id);
+            writer.WritePropertyName("result");
             if (value.Result != null)
             {
-                writer.WritePropertyName("result");
                 serializer.Serialize(writer, value.Result);
             }
+            else{
+                writer.WriteNull();
+            }
             writer.WriteEndObject();
         }
     }