From b8339c05a9e06f9cdf8e170fa5cbbb56fd0c224e Mon Sep 17 00:00:00 2001 From: XT <73406141+xtovski@users.noreply.github.com> Date: Fri, 11 Aug 2023 17:05:30 +0200 Subject: [PATCH 1/2] fix: added skip_serialization for JSON fields that should not be present if value is None --- json/jsonrpc/src/convention.rs | 2 ++ 1 file changed, 2 insertions(+) diff --git a/json/jsonrpc/src/convention.rs b/json/jsonrpc/src/convention.rs index 26e8f2c2..b62e0c6d 100644 --- a/json/jsonrpc/src/convention.rs +++ b/json/jsonrpc/src/convention.rs @@ -104,11 +104,13 @@ pub struct Response { /// This member is REQUIRED on success. /// This member MUST NOT exist if there was an error invoking the method. /// The value of this member is determined by the method invoked on the Server. + #[serde(skip_serializing_if = "Option::is_none")] pub result: Value, // This member is REQUIRED on error. // This member MUST NOT exist if there was no error triggered during invocation. // The value for this member MUST be an Object as defined in section 5.1. + #[serde(skip_serializing_if = "Option::is_none")] pub error: Option, /// This member is REQUIRED. From c74fc5c02b16cf5121802c2d7a9f92fef9983613 Mon Sep 17 00:00:00 2001 From: Rob Ede Date: Sun, 29 Oct 2023 23:26:36 +0000 Subject: [PATCH 2/2] Update json/jsonrpc/src/convention.rs --- json/jsonrpc/src/convention.rs | 1 - 1 file changed, 1 deletion(-) diff --git a/json/jsonrpc/src/convention.rs b/json/jsonrpc/src/convention.rs index b62e0c6d..2ef96b7d 100644 --- a/json/jsonrpc/src/convention.rs +++ b/json/jsonrpc/src/convention.rs @@ -104,7 +104,6 @@ pub struct Response { /// This member is REQUIRED on success. /// This member MUST NOT exist if there was an error invoking the method. /// The value of this member is determined by the method invoked on the Server. - #[serde(skip_serializing_if = "Option::is_none")] pub result: Value, // This member is REQUIRED on error.