Added Config Manager

Author: jamesgober

src/config.rs

@@ -23,7 +23,7 @@ pub const ENABLE_COMPRESSION: bool = false;
 pub const ENABLE_ENCRYPTION: bool = true;
 
 /// Main network configuration structure that contains all configurable settings
-#[derive(Debug, Clone, Deserialize, Serialize)]
+#[derive(Debug, Clone, Deserialize, Serialize, Default)]
 pub struct NetworkConfig {
     /// Server-specific configuration
     #[serde(default)]
@@ -42,34 +42,25 @@ pub struct NetworkConfig {
     pub logging: LoggingConfig,
 }
 
-impl Default for NetworkConfig {
-    fn default() -> Self {
-        Self {
-            server: ServerConfig::default(),
-            client: ClientConfig::default(),
-            transport: TransportConfig::default(),
-            logging: LoggingConfig::default(),
-        }
-    }
-}
+// Default implementation is now derived
 
 impl NetworkConfig {
     /// Load configuration from a TOML file
     pub fn from_file<P: AsRef<Path>>(path: P) -> Result<Self> {
         let mut file = File::open(path)
-            .map_err(|e| ProtocolError::ConfigError(format!("Failed to open config file: {}", e)))?;
+            .map_err(|e| ProtocolError::ConfigError(format!("Failed to open config file: {e}")))?;
 
         let mut contents = String::new();
         file.read_to_string(&mut contents)
-            .map_err(|e| ProtocolError::ConfigError(format!("Failed to read config file: {}", e)))?;
+            .map_err(|e| ProtocolError::ConfigError(format!("Failed to read config file: {e}")))?;
 
         Self::from_toml(&contents)
     }
 
     /// Load configuration from TOML string
     pub fn from_toml(content: &str) -> Result<Self> {
         toml::from_str::<Self>(content)
-            .map_err(|e| ProtocolError::ConfigError(format!("Failed to parse TOML: {}", e)))
+            .map_err(|e| ProtocolError::ConfigError(format!("Failed to parse TOML: {e}")))
     }
 
     /// Load configuration from environment variables
@@ -122,10 +113,10 @@ impl NetworkConfig {
     /// Save configuration to a file
     pub fn save_to_file<P: AsRef<Path>>(&self, path: P) -> Result<()> {
         let content = toml::to_string_pretty(self)
-            .map_err(|e| ProtocolError::ConfigError(format!("Failed to serialize config: {}", e)))?;
+            .map_err(|e| ProtocolError::ConfigError(format!("Failed to serialize config: {e}")))?;
 
         std::fs::write(path, content)
-            .map_err(|e| ProtocolError::ConfigError(format!("Failed to write config file: {}", e)))?;
+            .map_err(|e| ProtocolError::ConfigError(format!("Failed to write config file: {e}")))?;
 
         Ok(())
     }
@@ -328,6 +319,6 @@ mod log_level_serde {
     {
         let level_str = String::deserialize(deserializer)?;
         Level::from_str(&level_str)
-            .map_err(|_| serde::de::Error::custom(format!("Invalid log level: {}", level_str)))
+            .map_err(|_| serde::de::Error::custom(format!("Invalid log level: {level_str}")))
     }
 }

src/service/client.rs

@@ -26,8 +26,10 @@ impl Client {
     /// Connect and perform secure handshake with timeout using default configuration
     #[instrument(skip(addr), fields(address = %addr))]
     pub async fn connect(addr: &str) -> Result<Self> {
-        let mut config = ClientConfig::default();
-        config.address = addr.to_string();
+        let config = ClientConfig {
+            address: addr.to_string(),
+            ..Default::default()
+        };
         Self::connect_with_config(config).await
     }
 

src/service/daemon.rs

@@ -47,8 +47,10 @@ pub async fn start_with_shutdown(
     shutdown_rx: oneshot::Receiver<()>
 ) -> Result<()> {
     // Use default configuration with overridden address
-    let mut config = ServerConfig::default();
-    config.address = addr.to_string();
+    let config = ServerConfig {
+        address: addr.to_string(),
+        ..Default::default()
+    };
     start_with_config_and_shutdown(config, shutdown_rx).await
 }