added config validation to network requester

clients/native/src/commands/run.rs

@@ -113,7 +113,7 @@ pub(crate) async fn execute(args: &Run) -> Result<(), Box<dyn Error + Send + Syn
     };
 
     if !config.validate() {
-        return Err(Box::new(ClientError::ConfigValidationFailure))
+        return Err(Box::new(ClientError::ConfigValidationFailure));
     }
 
     let override_config_fields = OverrideConfig::from(args.clone());

clients/socks5/src/commands/run.rs

@@ -123,7 +123,7 @@ pub(crate) async fn execute(args: &Run) -> Result<(), Box<dyn std::error::Error
     };
 
     if !config.validate() {
-        return Err(Box::new(Socks5ClientError::ConfigValidationFailure))
+        return Err(Box::new(Socks5ClientError::ConfigValidationFailure));
     }
 
     let override_config_fields = OverrideConfig::from(args.clone());

service-providers/network-requester/src/cli/run.rs

@@ -126,6 +126,10 @@ pub(crate) async fn execute(args: &Run) -> Result<(), NetworkRequesterError> {
         }
     };
 
+    if !config.validate() {
+        return Err(NetworkRequesterError::ConfigValidationFailure);
+    }
+
     let override_config_fields = OverrideConfig::from(args.clone());
     config = override_config(config, override_config_fields);
 

service-providers/network-requester/src/config/mod.rs

@@ -119,6 +119,11 @@ impl Config {
         cfg
     }
 
+    pub fn validate(&self) -> bool {
+        // no other sections have explicit requirements (yet)
+        self.base.validate()
+    }
+
     // getters
     pub fn get_config_file_save_location(&self) -> PathBuf {
         self.config_directory().join(Self::config_file_name())

service-providers/network-requester/src/error.rs

@@ -21,6 +21,10 @@ pub enum NetworkRequesterError {
     #[error("failed to load configuration file: {0}")]
     FailedToLoadConfig(String),
 
+    // TODO: add more details here
+    #[error("Failed to validate the loaded config")]
+    ConfigValidationFailure,
+
     #[error("failed local version check, client and config mismatch")]
     FailedLocalVersionCheck,