fixing middleware fallthrough

Author: shakyShane

src/lib/presets/m2/handlers/build.rs

@@ -1,8 +1,8 @@
-use actix_web::http::StatusCode;
 use actix_web::HttpRequest;
 use actix_web::HttpResponse;
 use preset::AppState;
 use presets::m2::state::gather_state;
+use actix_web::http::StatusCode;
 
 ///
 /// This handler will serve up JSON that
@@ -25,13 +25,6 @@ pub fn handle(req: &HttpRequest<AppState>) -> HttpResponse {
 
     match output {
         Ok(t) => HttpResponse::Ok().content_type("application/json").body(t),
-        Err(e) => HttpResponse::Ok()
-            .content_type("application/json")
-            .status(StatusCode::from_u16(500).expect("can set 500 resp code"))
-            .body(
-                serde_json::to_string_pretty(&json!({
-                "message": e.to_string()
-            })).unwrap(),
-            ),
+        Err(e) => super::err_response::create(e),
     }
 }

src/lib/presets/m2/handlers/err_response.rs

@@ -0,0 +1,13 @@
+use actix_web::http::StatusCode;
+use actix_web::HttpResponse;
+
+pub fn create(message: String) -> HttpResponse {
+    HttpResponse::Ok()
+        .content_type("application/json")
+        .status(StatusCode::from_u16(500).expect("can set 500 resp code"))
+        .body(
+            serde_json::to_string_pretty(&json!({
+                "message": message
+            })).unwrap(),
+        )
+}

src/lib/presets/m2/handlers/loaders.rs

@@ -23,7 +23,7 @@ pub fn handle(req: &HttpRequest<AppState>) -> HttpResponse {
     };
 
     match output {
-        Ok(t) => HttpResponse::Ok().content_type("text/plain").body(t),
-        Err(_e) => HttpResponse::Ok().content_type("text/plain").body("NAH"),
+        Ok(t) => HttpResponse::Ok().content_type("application/javascript").body(t),
+        Err(e) => super::err_response::create(e),
     }
 }

src/lib/presets/m2/handlers/mod.rs

@@ -7,3 +7,4 @@ pub mod req_capture;
 pub mod requests;
 pub mod seed;
 pub mod serve_r_js;
+pub mod err_response;