config: improve config logging

Author: pingoo

.github/workflows/release.yml

@@ -16,7 +16,7 @@ permissions:
 
 env:
   DOCKER_IMAGE_GITHUB: ghcr.io/pingooio/pingoo
-  DOCKER_IMAGE: pingooio/pingoo
+  DOCKER_IMAGE: docker.io/pingooio/pingoo
 
 jobs:
   docker_build:

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "pingoo"
-version = "0.14.4"
+version = "0.14.5"
 edition = "2024"
 
 [[bin]]

Cargo.toml

@@ -10,7 +10,7 @@ use http::StatusCode;
 use indexmap::IndexMap;
 use serde::{Deserialize, Serialize};
 use tokio::fs;
-use tracing::{debug, info, warn};
+use tracing::{debug, warn};
 
 use crate::{
     Error,
@@ -200,8 +200,6 @@ pub async fn load_and_validate() -> Result<Config, Error> {
         .await
         .map_err(|err| Error::Config(format!("error reading config file ({DEFAULT_CONFIG_FILE}): {err}")))?;
 
-    info!("configuration successfully loaded from {DEFAULT_CONFIG_FILE}");
-
     let mut config_file: ConfigFile = serde_yaml::from_slice(&raw_config)
         .map_err(|err| Error::Config(format!("error parsing config file ({DEFAULT_CONFIG_FILE}): {err}")))?;
 

pingoo/config/config.rs

@@ -22,6 +22,7 @@ mod tls;
 pub use error::Error;
 
 use crate::{listeners::GRACEFUL_SHUTDOWN_TIMEOUT, server::Server};
+use config::DEFAULT_CONFIG_FILE;
 
 // We don't use mimalloc in debug builds to speedup compilation
 // #[cfg(not(debug_assertions))]
@@ -47,6 +48,11 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
         .map_err(|err| Error::Config(format!("error setting up rustls crypto provider: {err:?}")))?;
 
     let config = config::load_and_validate().await?;
+    info!(
+        services = config.services.len(),
+        listeners = config.listeners.len(),
+        "configuration successfully loaded from {DEFAULT_CONFIG_FILE}"
+    );
 
     let (shutdown_tx, shutdown_rx) = watch::channel(());
     tokio::spawn(shutdown_signal(shutdown_tx));

pingoo/main.rs

@@ -262,12 +262,12 @@ async fn try_metadata(path: &PathBuf) -> Result<Metadata, Response<BoxBody<Bytes
         Ok(metadata) if metadata.is_dir() => {
             debug!("error getting {path:?}: is a directory");
             Err(new_not_found_error())
-        },
+        }
         Ok(metadata) => Ok(metadata),
         Err(error) if error.kind() == std::io::ErrorKind::NotFound => {
             debug!("error getting {path:?}: not found");
             Err(new_not_found_error())
-        },
+        }
         Err(err) => {
             error!("error getting metadata for static file: {path:?}: {err}");
             Err(new_internal_error_response_500())