refactor: move directory listing initialization into the correspondin…

…g module (#373)

src/directory_listing.rs

@@ -20,7 +20,7 @@ use std::io;
 use std::path::Path;
 use std::time::{SystemTime, UNIX_EPOCH};
 
-use crate::{http_ext::MethodExt, Context, Result};
+use crate::{handler::RequestHandlerOpts, http_ext::MethodExt, Context, Result};
 
 /// Non-alphanumeric characters to be percent-encoded
 /// excluding the "unreserved characters" because allowed in a URI.
@@ -59,6 +59,21 @@ pub struct DirListOpts<'a> {
     pub ignore_hidden_files: bool,
 }
 
+/// 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}");
+
+    handler_opts.dir_listing_order = order;
+    server_info!("directory listing order code: {order}");
+
+    handler_opts.dir_listing_format = format;
+    server_info!(
+        "directory listing format: {:?}",
+        handler_opts.dir_listing_format
+    );
+}
+
 /// Provides directory listing support for the current request.
 /// Note that this function highly depends on `static_files::composed_file_metadata()` function
 /// which must be called first. See `static_files::handle()` for more details.

src/server.rs

@@ -26,6 +26,8 @@ use {
     hyper::service::{make_service_fn, service_fn},
 };
 
+#[cfg(feature = "directory-listing")]
+use crate::directory_listing;
 #[cfg(feature = "fallback-page")]
 use crate::fallback_page;
 
@@ -214,26 +216,6 @@ impl Server {
             );
         }
 
-        // Directory listing options
-        #[cfg(feature = "directory-listing")]
-        let dir_listing = general.directory_listing;
-        #[cfg(feature = "directory-listing")]
-        server_info!("directory listing: enabled={}", dir_listing);
-        // Directory listing order number
-        #[cfg(feature = "directory-listing")]
-        let dir_listing_order = general.directory_listing_order;
-        #[cfg(feature = "directory-listing")]
-        server_info!("directory listing order code: {}", dir_listing_order);
-        // Directory listing format
-        #[cfg(feature = "directory-listing")]
-        let dir_listing_format = general.directory_listing_format;
-        #[cfg(feature = "directory-listing")]
-        server_info!("directory listing format: {:?}", dir_listing_format);
-
-        // Cache control headers option
-        let cache_control_headers = general.cache_control_headers;
-        server_info!("cache control headers: enabled={}", cache_control_headers);
-
         // Log remote address option
         let log_remote_address = general.log_remote_address;
         server_info!("log remote address: enabled={}", log_remote_address);
@@ -267,12 +249,6 @@ impl Server {
         // Request handler options, some settings will be filled in by modules
         let mut handler_opts = RequestHandlerOpts {
             root_dir,
-            #[cfg(feature = "directory-listing")]
-            dir_listing,
-            #[cfg(feature = "directory-listing")]
-            dir_listing_order,
-            #[cfg(feature = "directory-listing")]
-            dir_listing_format,
             page404: page404.clone(),
             page50x: page50x.clone(),
             log_remote_address,
@@ -283,6 +259,15 @@ impl Server {
             ..Default::default()
         };
 
+        // Directory listing options
+        #[cfg(feature = "directory-listing")]
+        directory_listing::init(
+            general.directory_listing,
+            general.directory_listing_order,
+            general.directory_listing_format,
+            &mut handler_opts,
+        );
+
         // Fallback page option
         #[cfg(feature = "fallback-page")]
         fallback_page::init(&general.page_fallback, &mut handler_opts);