actix: Log an error if we have an internal server error

Signed-off-by: Ian Jackson <ijackson@chiark.greenend.org.uk>

diff --git a/daemon/api.rs b/daemon/api.rs
index 1337b7a783616423bd0f3060e93aabf3596b11cb..4ec910f645a1aa16674d236076f2df898ca4b18e 100644 (file)
--- a/daemon/api.rs
+++ b/daemon/api.rs
@@ -90,9 +90,7 @@ impl ResponseError for FatalErrorResponse {
   }
 
   fn error_response(&self) -> HttpResponse<BoxBody> {
-    self.status_code().respond_text(
-      &format_args!("Online-layer error\n{:?}\n{}\n", self, self)
-    )
+    error_response(self)
   }
 }
 

diff --git a/daemon/main.rs b/daemon/main.rs
index 9ea649e0051eab392d81d2d4013c2d7a79bb5d5c..fca559f0ac889e34f33e36b783b57c951432ce0c 100644 (file)
--- a/daemon/main.rs
+++ b/daemon/main.rs
@@ -317,6 +317,10 @@ impl ResponseError for BundleDownloadError {
       BDE::IE(_)               => StatusCode::INTERNAL_SERVER_ERROR,
     }
   }
+
+  fn error_response(&self) -> HttpResponse<BoxBody> {
+    error_response(self)
+  }
 }
 
 #[route("/_/bundle/{instance}/{id}", method="GET", method="HEAD")]
@@ -407,6 +411,18 @@ async fn not_found_handler(method: Method) -> impl Responder {
   }
 }
 
+fn error_response<E>(self_: &E) -> HttpResponse<BoxBody>
+where E: ResponseError + Debug + Display
+{
+  let status = self_.status_code();
+  if status == StatusCode::INTERNAL_SERVER_ERROR {
+    error!("responding with internal error -- {} -- {:?}", self_, self_);
+  }
+  self_.status_code().respond_text(
+    &format_args!("{}\n{:?}\n", self_, self_)
+  )
+}
+
 #[actix_web::main] // not compatible with fehler
 async fn main() -> Result<(),StartupError> {
   use structopt::StructOpt;