From b0050f61893de8bdc0a88be4099e4e286dfe7632 Mon Sep 17 00:00:00 2001
From: lovasoa <pere.jobs@gmail.com>
Date: Mon, 16 Oct 2023 00:12:35 +0200
Subject: [PATCH] better error logs on 404

---
 src/webserver/http.rs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/webserver/http.rs b/src/webserver/http.rs
index f9308ac8..2a7dd602 100644
--- a/src/webserver/http.rs
+++ b/src/webserver/http.rs
@@ -415,13 +415,14 @@ async fn process_sql_request(
         .sql_file_cache
         .get(app_state, &sql_path)
         .await
+        .with_context(|| format!("Unable to get SQL file {:?}", sql_path))
         .map_err(anyhow_err_to_actix)?;
     let response = render_sql(&mut req, sql_file).await?;
     Ok(req.into_response(response))
 }
 
 fn anyhow_err_to_actix(e: anyhow::Error) -> actix_web::Error {
-    log::error!("Error while trying to get SQL file: {:#}", e);
+    log::error!("{e:#}");
     match e.downcast::<ErrorWithStatus>() {
         Ok(err) => actix_web::Error::from(err),
         Err(e) => ErrorInternalServerError(format!(
@@ -442,6 +443,7 @@ async fn serve_file(
             .file_system
             .modified_since(state, path.as_ref(), since, false)
             .await
+            .with_context(|| format!("Unable to get modification time of file {:?}", path))
             .map_err(anyhow_err_to_actix)?;
         if !modified {
             return Ok(HttpResponse::NotModified().finish());
@@ -451,6 +453,7 @@ async fn serve_file(
         .file_system
         .read_file(state, path.as_ref(), false)
         .await
+        .with_context(|| format!("Unable to read file {:?}", path))
         .map_err(anyhow_err_to_actix)
         .map(|b| {
             HttpResponse::Ok()