feat(Modules): load modules configs before Log initialization (#6104)

in order to properly load modules appenders and loggers

1 files changed, 7 insertions, 8 deletions

diff --git a/src/common/Configuration/Config.cpp b/src/common/Configuration/Config.cpp
index 6bfc88e259..d24e082190 100644
--- a/src/common/Configuration/Config.cpp
+++ b/src/common/Configuration/Config.cpp
@@ -347,7 +347,6 @@ bool ConfigMgr::LoadModulesConfigs()
         return true;
 
     // Start loading module configs
-    std::vector<std::string /*config variant*/> moduleConfigFiles;
     std::string const& moduleConfigPath = GetConfigPath() + "modules/";
     bool isExistDefaultConfig = true;
     bool isExistDistConfig = true;
@@ -368,25 +367,25 @@ bool ConfigMgr::LoadModulesConfigs()
             isExistDefaultConfig = false;
 
         if (isExistDefaultConfig && isExistDistConfig)
-            moduleConfigFiles.emplace_back(defaultFileName);
+            _moduleConfigFiles.emplace_back(defaultFileName);
         else if (!isExistDefaultConfig && isExistDistConfig)
-            moduleConfigFiles.emplace_back(distFileName);
+            _moduleConfigFiles.emplace_back(distFileName);
     }
 
     // If module configs not exist - no load
-    if (moduleConfigFiles.empty())
-        return false;
+    return !_moduleConfigFiles.empty();
+}
 
+void ConfigMgr::PrintLoadedModulesConfigs()
+{
     // Print modules configurations
     LOG_INFO("server", " ");
     LOG_INFO("server", "Using modules configuration:");
 
-    for (auto const& itr : moduleConfigFiles)
+    for (auto const& itr : _moduleConfigFiles)
         LOG_INFO("server", "> %s", itr.c_str());
 
     LOG_INFO("server", " ");
-
-    return true;
 }
 
 /*