Merge pull request #168 from hyperware-ai/j/prevent-crashing

fix: prevent crashing on invalid error reponse

Author: jurij-jukic

src/hyperapp.rs

@@ -277,9 +277,16 @@ where
         return Ok(r);
     }
 
-    let e = serde_json::from_slice::<SendError>(&response_bytes)
-        .expect("Failed to deserialize response to send()");
-    return Err(AppSendError::SendError(e));
+    match serde_json::from_slice::<SendError>(&response_bytes) {
+        Ok(e) => Err(AppSendError::SendError(e)),
+        Err(err) => {
+            error!(
+                "Failed to deserialize response in send(): {} (payload: {:?})",
+                err, response_bytes
+            );
+            Err(AppSendError::BuildError(BuildError::NoBody))
+        }
+    }
 }
 
 pub async fn send_rmp<R>(request: Request) -> Result<R, AppSendError>
@@ -302,9 +309,16 @@ where
         return Ok(r);
     }
 
-    let e = rmp_serde::from_slice::<SendError>(&response_bytes)
-        .expect("Failed to deserialize response to send()");
-    return Err(AppSendError::SendError(e));
+    match rmp_serde::from_slice::<SendError>(&response_bytes) {
+        Ok(e) => Err(AppSendError::SendError(e)),
+        Err(err) => {
+            error!(
+                "Failed to deserialize response in send_rmp(): {} (payload: {:?})",
+                err, response_bytes
+            );
+            Err(AppSendError::BuildError(BuildError::NoBody))
+        }
+    }
 }
 
 // Enum defining the state persistance behaviour