fix: generic server log info output even on higher log levels (#542)

It fixes an issue (previously an ad-hoc and inconsistent behavior) where SWS printed INFO-level messages at startup despite setting up a higher log level. E.g., `WARN` or `ERROR`.

Notice:

For consistency reasons, SWS now won't persistently show server information at startup independently of the log level as it did before. Instead, those info log entries are non-persistent and under the normal `INFO` log level.

For example, to show those logs again then use the `INFO` log level.

$ static-web-server -p 8787 -d public/ -g info

Author: joseluisq

src/basic_auth.rs

@@ -15,7 +15,7 @@ use crate::{error_page, handler::RequestHandlerOpts, http_ext::MethodExt, Error}
 /// Initializes `Basic` HTTP Authorization handling
 pub(crate) fn init(credentials: &str, handler_opts: &mut RequestHandlerOpts) {
     credentials.trim().clone_into(&mut handler_opts.basic_auth);
-    server_info!(
+    tracing::info!(
         "basic authentication: enabled={}",
         !handler_opts.basic_auth.is_empty()
     );

src/bin/server.rs

@@ -25,8 +25,6 @@ fn main() -> Result {
     }
 
     if let Some(commands) = opts.general.commands {
-        use static_web_server::server_info;
-
         match commands {
             #[cfg(windows)]
             Commands::Install {} => {
@@ -45,13 +43,13 @@ fn main() -> Result {
                     let mut comp_dir = out_dir.clone();
                     comp_dir.push("completions");
                     clap_allgen::render_shell_completions::<General>(&comp_dir)?;
-                    server_info!("wrote completions to {}", comp_dir.to_string_lossy());
+                    tracing::info!("wrote completions to {}", comp_dir.to_string_lossy());
                 }
                 if man_pages || !completions {
                     let mut man_dir = out_dir.clone();
                     man_dir.push("man");
                     clap_allgen::render_manpages::<General>(&man_dir)?;
-                    server_info!("wrote man pages to {}", man_dir.to_string_lossy());
+                    tracing::info!("wrote man pages to {}", man_dir.to_string_lossy());
                 }
                 return Ok(());
             }

src/compression.rs

@@ -82,7 +82,7 @@ pub fn init(enabled: bool, level: CompressionLevel, handler_opts: &mut RequestHa
         #[cfg(any(feature = "compression", feature = "compression-zstd"))]
         "zstd",
     ];
-    server_info!(
+    tracing::info!(
         "auto compression: enabled={enabled}, formats={}, compression level={level:?}",
         FORMATS.join(",")
     );

src/compression_static.rs

@@ -31,7 +31,7 @@ pub struct CompressedFileVariant {
 /// Initializes static compression.
 pub fn init(enabled: bool, handler_opts: &mut RequestHandlerOpts) {
     handler_opts.compression_static = enabled;
-    server_info!("compression static: enabled={enabled}");
+    tracing::info!("compression static: enabled={enabled}");
 }
 
 /// Post-processing to add Vary header if necessary.

src/control_headers.rs

@@ -26,7 +26,7 @@ const CACHE_EXT_ONE_YEAR: [&str; 32] = [
 
 pub(crate) fn init(enabled: bool, handler_opts: &mut RequestHandlerOpts) {
     handler_opts.cache_control_headers = enabled;
-    server_info!("cache control headers: enabled={enabled}");
+    tracing::info!("cache control headers: enabled={enabled}");
 }
 
 /// Appends `Cache-Control` header to a response if necessary

src/cors.rs

@@ -71,7 +71,7 @@ pub fn new(
         };
 
         if cors_res.is_some() {
-            server_info!(
+            tracing::info!(
                     "cors enabled=true, allow_methods=[GET,HEAD,OPTIONS], allow_origins={}, allow_headers=[{}], expose_headers=[{}]",
                     origins_str,
                     allow_headers_str,

src/directory_listing.rs

@@ -61,13 +61,13 @@ pub struct DirListOpts<'a> {
 /// Initializes directory listings.
 pub fn init(enabled: bool, order: u8, format: DirListFmt, handler_opts: &mut RequestHandlerOpts) {
     handler_opts.dir_listing = enabled;
-    server_info!("directory listing: enabled={enabled}");
+    tracing::info!("directory listing: enabled={enabled}");
 
     handler_opts.dir_listing_order = order;
-    server_info!("directory listing order code: {order}");
+    tracing::info!("directory listing order code: {order}");
 
     handler_opts.dir_listing_format = format;
-    server_info!(
+    tracing::info!(
         "directory listing format: {:?}",
         handler_opts.dir_listing_format
     );

src/fallback_page.rs

@@ -26,7 +26,7 @@ pub(crate) fn init(file_path: &Path, handler_opts: &mut RequestHandlerOpts) {
         tracing::debug!("fallback page path not found or not a regular file");
     }
 
-    server_info!(
+    tracing::info!(
         "fallback page: enabled={}, value=\"{}\"",
         found,
         file_path.display()

src/health.rs

@@ -14,7 +14,7 @@ use crate::{handler::RequestHandlerOpts, Error};
 /// Initializes the health endpoint.
 pub fn init(enabled: bool, handler_opts: &mut RequestHandlerOpts) {
     handler_opts.health = enabled;
-    server_info!("health endpoint: enabled={enabled}");
+    tracing::info!("health endpoint: enabled={enabled}");
 }
 
 /// Handles health requests.

src/log_addr.rs

@@ -20,16 +20,16 @@ pub(crate) fn init(enabled: bool, handler_opts: &mut RequestHandlerOpts) {
         format!("{:?}", handler_opts.trusted_proxies)
     };
 
-    server_info!("log requests with remote IP addresses: enabled={enabled}");
-    server_info!(
+    tracing::info!("log requests with remote IP addresses: enabled={enabled}");
+    tracing::info!(
         "log X-Real-IP header: enabled={}",
         handler_opts.log_forwarded_for
     );
-    server_info!(
+    tracing::info!(
         "log X-Forwarded-For header: enabled={}",
         handler_opts.log_forwarded_for
     );
-    server_info!("trusted IPs for X-Forwarded-For: {trusted}");
+    tracing::info!("trusted IPs for X-Forwarded-For: {trusted}");
 }
 
 /// It logs remote and real IP addresses if available.