Bugfix - Visibly warn if failed to deserialize websocket response. ()

This commit is contained in:
Finn Bear 2023-05-16 04:31:16 -07:00 committed by GitHub
parent a8f35611c2
commit 437fc231f1
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 5 additions and 11 deletions
lib/src/api/engine/remote

View file

@ -107,7 +107,7 @@ pub(crate) fn router(base_url: Url, client: reqwest::Client, route_rx: Receiver<
let mut stream = route_rx.into_stream();
while let Some(Some(route)) = stream.next().await {
match super::router(
let result = super::router(
route.request,
&base_url,
&client,
@ -115,15 +115,8 @@ pub(crate) fn router(base_url: Url, client: reqwest::Client, route_rx: Receiver<
&mut vars,
&mut auth,
)
.await
{
Ok(value) => {
let _ = route.response.into_send_async(Ok(value)).await;
}
Err(error) => {
let _ = route.response.into_send_async(Err(error)).await;
}
}
.await;
let _ = route.response.into_send_async(result).await;
}
});
}

View file

@ -306,7 +306,8 @@ pub(crate) fn router(
}
}
Err(_error) => {
trace!(target: LOG, "Failed to deserialise message");
// Unfortunately, we don't know which response failed to deserialize
warn!(target: LOG, "Failed to deserialise message");
}
},
Err(error) => {