fix: run custom validation on config init or reload

Author: jescalada

index.ts

@@ -48,7 +48,7 @@ if (argv.v) {
   process.exit(0);
 }
 
-console.log('validating config');
+console.log('Validating config');
 validate();
 
 console.log('Setting up the proxy and Service');

src/config/ConfigLoader.ts

@@ -7,6 +7,7 @@ import { EventEmitter } from 'events';
 import envPaths from 'env-paths';
 import { GitProxyConfig, Convert } from './generated/config';
 import { Configuration, ConfigurationSource, FileSource, HttpSource, GitSource } from './types';
+import { validateConfig } from '.';
 
 const execFileAsync = promisify(execFile);
 
@@ -189,6 +190,11 @@ export class ConfigLoader extends EventEmitter {
       } else {
         console.log('Configuration has not changed, no update needed');
       }
+
+      if (!validateConfig(newConfig)) {
+        console.error('Invalid configuration, skipping reload');
+        return;
+      }
     } catch (error: any) {
       console.error('Error reloading configuration:', error);
       this.emit('configurationError', error);

src/config/index.ts

@@ -1,7 +1,7 @@
 import { existsSync, readFileSync } from 'fs';
 
 import defaultSettings from '../../proxy.config.json';
-import { GitProxyConfig, Convert } from './generated/config';
+import { GitProxyConfig, Convert, CommitConfig } from './generated/config';
 import { ConfigLoader } from './ConfigLoader';
 import { Configuration } from './types';
 import { serverConfig } from './env';
@@ -69,9 +69,26 @@ function loadFullConfiguration(): GitProxyConfig {
 
   _currentConfig = mergeConfigurations(defaultConfig, userSettings);
 
+  if (!validateConfig(_currentConfig)) {
+    console.error(
+      'Invalid configuration: Please check your configuration file and restart GitProxy.',
+    );
+    process.exit(1);
+  }
+
   return _currentConfig;
 }
 
+/**
+ * Performs additional custom validations on the configuration
+ *
+ * @param config The configuration to validate
+ * @returns true if the configuration is valid, false otherwise
+ */
+export const validateConfig = (config: GitProxyConfig): boolean => {
+  return validateCommitConfig(config.commitConfig ?? {});
+};
+
 /**
  * Merge configurations with environment variable overrides
  * @param {GitProxyConfig} defaultConfig - The default configuration