chore: disable metrics (#280)

decentraland · Oct 18, 2023 · e33aa4b · e33aa4b
1 parent b81f7a1
commit e33aa4b
Show file tree

Hide file tree

Showing 2 changed files with 49 additions and 49 deletions.
diff --git a/src/api/app.rs b/src/api/app.rs
@@ -64,10 +64,10 @@ pub fn get_app_router(
     App::new()
         .app_data(data.clone())
         .wrap(TracingLogger::default())
-        .wrap(initialize_metrics(data.config.env.clone()))
-        .wrap(CheckMetricsToken::new(
-            data.config.wkc_metrics_bearer_token.clone(),
-        ))
+        // .wrap(initialize_metrics(data.config.env.clone()))
+        // .wrap(CheckMetricsToken::new(
+        //     data.config.wkc_metrics_bearer_token.clone(),
+        // ))
         .wrap(CheckAuthToken::new(protected_routes))
         .wrap(middleware::NormalizePath::trim())
         .service(live)

diff --git a/tests/route_metrics.rs b/tests/route_metrics.rs
@@ -3,70 +3,70 @@ pub use common::*;
 
 use actix_web::test;
 
-const METRICS_URI: &str = "/metrics";
+// const METRICS_URI: &str = "/metrics";
 
-#[actix_web::test]
-async fn metrics_endpoint_should_work() {
-    let mut config = get_configuration().await;
-    config.wkc_metrics_bearer_token = String::from("TEST_TOKEN");
-    let token = config.wkc_metrics_bearer_token.clone();
+// #[actix_web::test]
+// async fn metrics_endpoint_should_work() {
+//     let mut config = get_configuration().await;
+//     config.wkc_metrics_bearer_token = String::from("TEST_TOKEN");
+//     let token = config.wkc_metrics_bearer_token.clone();
 
-    let app = test::init_service(get_app(config, None).await).await;
+//     let app = test::init_service(get_app(config, None).await).await;
 
-    let header = ("authorization", format!("Bearer {token}"));
+//     let header = ("authorization", format!("Bearer {token}"));
 
-    let req = test::TestRequest::get()
-        .uri(METRICS_URI)
-        .insert_header(header)
-        .to_request();
+//     let req = test::TestRequest::get()
+//         .uri(METRICS_URI)
+//         .insert_header(header)
+//         .to_request();
 
-    let response = test::call_service(&app, req).await;
+//     let response = test::call_service(&app, req).await;
 
-    assert!(response.status().is_success());
-}
+//     assert!(response.status().is_success());
+// }
 
-#[actix_web::test]
-async fn metrics_endpoint_should_fail_401_when_no_token() {
-    let mut config = get_configuration().await;
-    config.wkc_metrics_bearer_token = String::from("TEST_TOKEN");
+// #[actix_web::test]
+// async fn metrics_endpoint_should_fail_401_when_no_token() {
+//     let mut config = get_configuration().await;
+//     config.wkc_metrics_bearer_token = String::from("TEST_TOKEN");
 
-    let app = test::init_service(get_app(config, None).await).await;
+//     let app = test::init_service(get_app(config, None).await).await;
 
-    let header = ("authorization", format!("Bearer {}", ""));
+//     let header = ("authorization", format!("Bearer {}", ""));
 
-    let req = test::TestRequest::get()
-        .uri(METRICS_URI)
-        .insert_header(header)
-        .to_request();
+//     let req = test::TestRequest::get()
+//         .uri(METRICS_URI)
+//         .insert_header(header)
+//         .to_request();
 
-    let response = test::call_service(&app, req).await;
+//     let response = test::call_service(&app, req).await;
 
-    assert_eq!(response.status(), 401)
-}
+//     assert_eq!(response.status(), 401)
+// }
 
-#[actix_web::test]
-async fn metrics_endpoint_should_fail_401_when_no_header() {
-    let mut config = get_configuration().await;
-    config.wkc_metrics_bearer_token = String::from("TEST_TOKEN");
+// #[actix_web::test]
+// async fn metrics_endpoint_should_fail_401_when_no_header() {
+//     let mut config = get_configuration().await;
+//     config.wkc_metrics_bearer_token = String::from("TEST_TOKEN");
 
-    let app = test::init_service(get_app(config, None).await).await;
+//     let app = test::init_service(get_app(config, None).await).await;
 
-    let req = test::TestRequest::get().uri(METRICS_URI).to_request();
+//     let req = test::TestRequest::get().uri(METRICS_URI).to_request();
 
-    let response = test::call_service(&app, req).await;
+//     let response = test::call_service(&app, req).await;
 
-    assert_eq!(response.status(), 401)
-}
+//     assert_eq!(response.status(), 401)
+// }
 
-#[actix_web::test]
-async fn metrics_endpoint_should_fail_500() {
-    let config = get_configuration().await;
+// #[actix_web::test]
+// async fn metrics_endpoint_should_fail_500() {
+//     let config = get_configuration().await;
 
-    let app = test::init_service(get_app(config, None).await).await;
+//     let app = test::init_service(get_app(config, None).await).await;
 
-    let req = test::TestRequest::get().uri(METRICS_URI).to_request();
+//     let req = test::TestRequest::get().uri(METRICS_URI).to_request();
 
-    let response = test::call_service(&app, req).await;
+//     let response = test::call_service(&app, req).await;
 
-    assert_eq!(response.status(), 500)
-}
+//     assert_eq!(response.status(), 500)
+// }