Don't always set env-var for logging

Author: shakyShane

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

@@ -27,14 +27,13 @@ pub fn handle(req: &HttpRequest<AppState>) -> FutResp {
                     mutex.config = next_config.config;
                     mutex.paths = next_config.paths;
                     mutex.shim = next_config.shim;
-                    "Was Good!".to_string()
-                }
-                Err(e) => e.to_string(),
-            };
 
-            Ok(HttpResponse::Ok()
-                .content_type("application/json")
-                .body("yo!"))
+                    Ok(HttpResponse::Ok()
+                        .content_type("application/json")
+                        .body("ok"))
+                }
+                Err(e) => Ok(super::err_response::create(e.to_string())),
+            }
         })
         .responder()
 }

src/lib/presets/m2/preset_m2.rs

@@ -80,7 +80,7 @@ impl Preset<AppState> for M2Preset {
 
         http_async_responders
             .into_iter()
-            .fold(app, |acc_app, (path, method, handle)| {
+            .fold(app, |acc_app, (path, _method, handle)| {
                 acc_app.resource(&path, move |r| r.f(handle))
             })
     }

src/main.rs

@@ -36,7 +36,6 @@ fn run_with_opts(opts: ProgramOptions) -> Result<(), ProgramStartError> {
     //
     // Logging config
     //
-    ::std::env::set_var("RUST_LOG", "actix_web=info");
     env_logger::init();
 
     //
@@ -96,7 +95,7 @@ fn run_with_opts(opts: ProgramOptions) -> Result<(), ProgramStartError> {
         .shutdown_timeout(0)
         .start();
 
-    println!("Started server: {}://{}", server_opts.scheme, local_addr);
+    println!("{}://{}", server_opts.scheme, local_addr);
 
     let _ = sys.run();