Skip to content

Commit

Permalink
Don't log anything when the remote logging stream errors
Browse files Browse the repository at this point in the history
The way things are this can happen normally during shutdown.
  • Loading branch information
bobbobbio committed Sep 5, 2024
1 parent f3a2b15 commit 4b45e08
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions crates/maelstrom-client/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -241,9 +241,7 @@ impl Client {
.await?
.into_inner();
tokio::task::spawn(async move {
if let Err(e) = handle_log_messages(&rpc_log, log_stream).await {
slog::error!(&rpc_log, "remote logging failed"; "error" => ?e);
}
let _ = handle_log_messages(&rpc_log, log_stream).await;
});
Ok(tonic::Response::new(proto::Void {}))
})?;
Expand Down

0 comments on commit 4b45e08

Please sign in to comment.