chore: removing /info endpoint in a favor of /health (#291)

Author: geekbrother

src/handlers/health.rs

@@ -5,8 +5,19 @@ use {
 };
 
 pub async fn handler(ExtractState(state): ExtractState<Arc<AppState>>) -> impl IntoResponse {
+    let build_commit = match state.build_info.version_control.clone() {
+        Some(v) => v.git().unwrap().commit_short_id.clone(),
+        None => String::new(),
+    };
     (
         StatusCode::OK,
-        format!("OK, echo-server v{}", state.build_info.crate_info.version,),
+        format!(
+            "OK v{}, commit hash: {}, features: {:?}, instance id: {}, uptime: {} seconds",
+            state.build_info.crate_info.version,
+            build_commit,
+            state.build_info.crate_info.enabled_features,
+            state.instance_id,
+            state.uptime.elapsed().as_secs(),
+        ),
     )
 }

src/handlers/info.rs

@@ -29,7 +29,6 @@ use {
 
 // Push
 pub mod delete_client;
-pub mod info;
 pub mod metrics;
 pub mod push_message;
 pub mod register_client;

src/handlers/mod.rs

@@ -235,7 +235,6 @@ pub async fn bootstap(mut shutdown: broadcast::Receiver<()>, config: Config) ->
 
         let app = Router::new()
             .route("/health", get(handlers::health::handler))
-            .route("/info", get(handlers::info::handler))
             .nest("/tenants", tenancy_routes)
             .route(
                 "/:tenant_id/clients",
@@ -263,7 +262,6 @@ pub async fn bootstap(mut shutdown: broadcast::Receiver<()>, config: Config) ->
     #[cfg(not(feature = "multitenant"))]
     let app = Router::new()
         .route("/health", get(handlers::health::handler))
-        .route("/info", get(handlers::info::handler))
         .route(
             "/clients",
             post(handlers::single_tenant_wrappers::register_handler),

src/lib.rs

@@ -53,6 +53,10 @@ pub struct AppState {
     pub public_ip: Option<IpAddr>,
     is_multitenant: bool,
     pub geoblock: Option<GeoBlockLayer<Arc<MaxMindResolver>>>,
+    /// Service instance identifier
+    pub instance_id: uuid::Uuid,
+    /// Service instance uptime measurement
+    pub uptime: std::time::Instant,
 }
 
 build_info::build_info!(fn build_info);
@@ -100,6 +104,8 @@ pub fn new_state(
         public_ip,
         is_multitenant,
         geoblock: None,
+        instance_id: uuid::Uuid::new_v4(),
+        uptime: std::time::Instant::now(),
     })
 }