From 8a79fbea235ef84039957c1500e3e12466bc6080 Mon Sep 17 00:00:00 2001
From: Jose Celano <josecelano@gmail.com>
Date: Tue, 8 Aug 2023 10:22:12 +0100
Subject: [PATCH] refactor: [#361] extract mod for configuration

---
 src/bootstrap/app.rs    | 31 +------------------------------
 src/bootstrap/config.rs | 37 +++++++++++++++++++++++++++++++++++++
 src/bootstrap/mod.rs    |  1 +
 3 files changed, 39 insertions(+), 30 deletions(-)
 create mode 100644 src/bootstrap/config.rs

diff --git a/src/bootstrap/app.rs b/src/bootstrap/app.rs
index c0e688a0..6961e15f 100644
--- a/src/bootstrap/app.rs
+++ b/src/bootstrap/app.rs
@@ -11,11 +11,11 @@
 //! 2. Initialize static variables.
 //! 3. Initialize logging.
 //! 4. Initialize the domain tracker.
-use std::env;
 use std::sync::Arc;
 
 use torrust_tracker_configuration::Configuration;
 
+use super::config::initialize_configuration;
 use crate::bootstrap;
 use crate::shared::clock::static_time;
 use crate::shared::crypto::ephemeral_instance_keys;
@@ -55,35 +55,6 @@ pub fn initialize_static() {
     lazy_static::initialize(&ephemeral_instance_keys::RANDOM_SEED);
 }
 
-/// It loads the application configuration from the environment.
-///
-/// There are two methods to inject the configuration:
-///
-/// 1. By using a config file: `config.toml`. The file must be in the same folder where you are running the tracker.
-/// 2. Environment variable: `TORRUST_TRACKER_CONFIG`. The variable contains the same contents as the `config.toml` file.
-///
-/// Environment variable has priority over the config file.
-///
-/// Refer to the [configuration documentation](https://docs.rs/torrust-tracker-configuration) for the configuration options.
-///
-/// # Panics
-///
-/// Will panic if it can't load the configuration from either
-/// `./config.toml` file or the env var `TORRUST_TRACKER_CONFIG`.
-#[must_use]
-fn initialize_configuration() -> Configuration {
-    const CONFIG_PATH: &str = "./config.toml";
-    const CONFIG_ENV_VAR_NAME: &str = "TORRUST_TRACKER_CONFIG";
-
-    if env::var(CONFIG_ENV_VAR_NAME).is_ok() {
-        println!("Loading configuration from env var {CONFIG_ENV_VAR_NAME}");
-        Configuration::load_from_env_var(CONFIG_ENV_VAR_NAME).unwrap()
-    } else {
-        println!("Loading configuration from config file {CONFIG_PATH}");
-        Configuration::load_from_file(CONFIG_PATH).unwrap()
-    }
-}
-
 /// It builds the domain tracker
 ///
 /// The tracker is the domain layer service. It's the entrypoint to make requests to the domain layer.
diff --git a/src/bootstrap/config.rs b/src/bootstrap/config.rs
new file mode 100644
index 00000000..187fddd0
--- /dev/null
+++ b/src/bootstrap/config.rs
@@ -0,0 +1,37 @@
+//! Initialize configuration from file or env var.
+//!
+//! All environment variables are prefixed with `TORRUST_TRACKER_BACK_`.
+use std::env;
+
+use torrust_tracker_configuration::Configuration;
+
+// Environment variables
+
+const CONFIG_PATH: &str = "./config.toml";
+const CONFIG_ENV_VAR_NAME: &str = "TORRUST_TRACKER_CONFIG";
+
+/// It loads the application configuration from the environment.
+///
+/// There are two methods to inject the configuration:
+///
+/// 1. By using a config file: `config.toml`. The file must be in the same folder where you are running the tracker.
+/// 2. Environment variable: `TORRUST_TRACKER_CONFIG`. The variable contains the same contents as the `config.toml` file.
+///
+/// Environment variable has priority over the config file.
+///
+/// Refer to the [configuration documentation](https://docs.rs/torrust-tracker-configuration) for the configuration options.
+///
+/// # Panics
+///
+/// Will panic if it can't load the configuration from either
+/// `./config.toml` file or the env var `TORRUST_TRACKER_CONFIG`.
+#[must_use]
+pub fn initialize_configuration() -> Configuration {
+    if env::var(CONFIG_ENV_VAR_NAME).is_ok() {
+        println!("Loading configuration from env var {CONFIG_ENV_VAR_NAME}");
+        Configuration::load_from_env_var(CONFIG_ENV_VAR_NAME).unwrap()
+    } else {
+        println!("Loading configuration from config file {CONFIG_PATH}");
+        Configuration::load_from_file(CONFIG_PATH).unwrap()
+    }
+}
diff --git a/src/bootstrap/mod.rs b/src/bootstrap/mod.rs
index e39cf3ad..22044aaf 100644
--- a/src/bootstrap/mod.rs
+++ b/src/bootstrap/mod.rs
@@ -6,5 +6,6 @@
 //! like cleaning torrents, and other jobs because they can be enabled/disabled depending on the configuration.
 //! For example, you can have more than one UDP and HTTP tracker, each server is executed like a independent job.
 pub mod app;
+pub mod config;
 pub mod jobs;
 pub mod logging;