chore: Minor tweaks to the HttpRequest definitions

For better serialization behaviour.
wasmerio · Jun 4, 2024 · 09e4630 · 09e4630
1 parent 6775a48
commit 09e4630
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/lib/config/src/app/http.rs b/lib/config/src/app/http.rs
@@ -13,6 +13,7 @@ pub struct HttpRequest {
     pub method: Option<String>,
 
     /// Request body as a string.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub body: Option<String>,
 
     /// Request timeout.
@@ -21,6 +22,7 @@ pub struct HttpRequest {
     #[serde(skip_serializing_if = "Option::is_none")]
     pub timeout: Option<String>,
 
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub expect: Option<HttpRequestExpect>,
 }
 
@@ -30,8 +32,8 @@ pub struct HttpRequest {
 )]
 pub struct HttpRequestExpect {
     /// Expected HTTP status codes.
-    #[serde(skip_serializing_if = "Vec::is_empty")]
-    pub status_codes: Vec<u16>,
+    #[serde(skip_serializing_if = "Option::is_none")]
+    pub status_codes: Option<Vec<u16>>,
 
     /// Text that must be present in the response body.
     #[serde(skip_serializing_if = "Option::is_none")]

diff --git a/lib/config/src/app/mod.rs b/lib/config/src/app/mod.rs
@@ -193,6 +193,7 @@ pub struct AppConfigCapabilityMapV1 {
     pub memory: Option<AppConfigCapabilityMemoryV1>,
 
     /// Enables app bootstrapping with startup snapshots.
+    #[serde(skip_serializing_if = "Option::is_none")]
     pub instaboot: Option<AppConfigCapabilityInstaBootV1>,
 }