Skip to content

Commit 172ce7b

Browse files
authored
Merge pull request #87 from flashbots/peg/rm-unwraps
Remove unwraps in file server and health check server
2 parents 3cca29f + a75a96c commit 172ce7b

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

src/file_server.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -42,8 +42,7 @@ pub(crate) async fn static_file_server(path: PathBuf) -> Result<SocketAddr, Prox
4242
tracing::info!("Statically serving {path:?} on {addr}");
4343

4444
tokio::spawn(async move {
45-
if let Err(err) = axum::serve(tokio::net::TcpListener::bind(addr).await.unwrap(), app).await
46-
{
45+
if let Err(err) = axum::serve(listener, app).await {
4746
tracing::error!("HTTP file server: {err}");
4847
}
4948
});

src/health_check.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,9 @@ pub async fn server(listen_addr: SocketAddr) -> anyhow::Result<SocketAddr> {
3131
tracing::info!("Starting health check server at {}", listen_addr);
3232

3333
tokio::spawn(async move {
34-
axum::serve(listener, app).await.unwrap();
34+
if let Err(err) = axum::serve(listener, app).await {
35+
tracing::error!("Health check server closed: {err}");
36+
}
3537
});
3638

3739
Ok(listen_addr)

0 commit comments

Comments
 (0)