remove static modelName

FISCO-BCOS · Oct 22, 2024 · aba75fd · aba75fd
1 parent 6b09c39
commit aba75fd
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 18 deletions.
diff --git a/bcos-boostssl/context/Common.h b/bcos-boostssl/context/Common.h
@@ -25,7 +25,7 @@
 
 
 #define CONTEXT_LOG(LEVEL) BCOS_LOG(LEVEL) << LOG_BADGE(m_moduleName) << "[BOOSTSSL][CTX]"
-#define NODEINFO_LOG(LEVEL) BCOS_LOG(LEVEL) << LOG_BADGE(m_moduleName) << "[BOOSTSSL][NODEINFO]"
+#define NODEINFO_LOG(LEVEL) BCOS_LOG(LEVEL) << "[BOOSTSSL][NODEINFO]"
 
 namespace bcos
 {  // namespace bcos

diff --git a/bcos-boostssl/context/NodeInfoTools.h b/bcos-boostssl/context/NodeInfoTools.h
@@ -29,8 +29,6 @@ namespace boostssl
 {
 namespace context
 {
-static std::string m_moduleName = "DEFAULT";
-
 class NodeInfoTools
 {
 public:
@@ -42,9 +40,6 @@ class NodeInfoTools
 
     static std::function<bool(bool, boost::asio::ssl::verify_context&)> newVerifyCallback(
         std::shared_ptr<std::string> nodeIDOut);
-
-    static std::string moduleName() { return m_moduleName; }
-    static void setModuleName(std::string _moduleName) { m_moduleName = _moduleName; }
 };
 
 }  // namespace context

diff --git a/bcos-boostssl/websocket/Common.h b/bcos-boostssl/websocket/Common.h
@@ -26,7 +26,7 @@
 #include <boost/beast/websocket.hpp>
 
 #define BOOST_SSL_LOG(LEVEL) BCOS_LOG(LEVEL) << "[BOOSTSSL]"
-#define WEBSOCKET_TOOL(LEVEL) BCOS_LOG(LEVEL) << LOG_BADGE(m_moduleName) << "[WS][TOOL]"
+#define WEBSOCKET_TOOL(LEVEL) BCOS_LOG(LEVEL) << "[WS][TOOL]"
 #define WEBSOCKET_CONNECTOR(LEVEL) BCOS_LOG(LEVEL) << LOG_BADGE(m_moduleName) << "[WS][CONNECTOR]"
 #define WEBSOCKET_VERSION(LEVEL) BCOS_LOG(LEVEL) << LOG_BADGE(m_moduleName) << "[WS][VERSION]"
 #define WEBSOCKET_SESSION(LEVEL) BCOS_LOG(LEVEL) << LOG_BADGE(m_moduleName) << "[WS][SESSION]"

diff --git a/bcos-boostssl/websocket/WsInitializer.cpp b/bcos-boostssl/websocket/WsInitializer.cpp
@@ -44,7 +44,6 @@ using namespace bcos::boostssl::http;
 void WsInitializer::initWsService(WsService::Ptr _wsService)
 {
     std::shared_ptr<WsConfig> _config = m_config;
-    std::string m_moduleName = _config->moduleName();
     auto messageFactory = m_messageFactory;
     if (!messageFactory)
     {
@@ -76,9 +75,7 @@ void WsInitializer::initWsService(WsService::Ptr _wsService)
     auto threadPool = std::make_shared<ThreadPool>("t_ws_pool", threadPoolSize);
 
     // init module_name for log
-    WsTools::setModuleName(m_moduleName);
-    NodeInfoTools::setModuleName(m_moduleName);
-    connector->setModuleName(m_moduleName);
+    connector->setModuleName(_config->moduleName());
 
     std::shared_ptr<boost::asio::ssl::context> srvCtx = nullptr;
     std::shared_ptr<boost::asio::ssl::context> clientCtx = nullptr;
@@ -87,8 +84,8 @@ void WsInitializer::initWsService(WsService::Ptr _wsService)
         auto contextBuilder = std::make_shared<ContextBuilder>();
 
         // init module_name for log
-        contextBuilder->setModuleName(m_moduleName);
-        _config->contextConfig()->setModuleName(m_moduleName);
+        contextBuilder->setModuleName(_config->moduleName());
+        _config->contextConfig()->setModuleName(_config->moduleName());
 
         srvCtx = contextBuilder->buildSslContext(true, *_config->contextConfig());
         clientCtx = contextBuilder->buildSslContext(false, *_config->contextConfig());
@@ -114,7 +111,7 @@ void WsInitializer::initWsService(WsService::Ptr _wsService)
 
         auto httpServerFactory = std::make_shared<HttpServerFactory>();
         auto httpServer = httpServerFactory->buildHttpServer(_config->listenIP(),
-            _config->listenPort(), ioServicePool->getIOService(), srvCtx, m_moduleName);
+            _config->listenPort(), ioServicePool->getIOService(), srvCtx, _config->moduleName());
         httpServer->setIOServicePool(ioServicePool);
         httpServer->setDisableSsl(_config->disableSsl());
         httpServer->setThreadPool(threadPool);

diff --git a/bcos-boostssl/websocket/WsTools.h b/bcos-boostssl/websocket/WsTools.h
@@ -34,7 +34,6 @@ namespace boostssl
 {
 namespace ws
 {
-static std::string m_moduleName = "DEFAULT";
 class WsTools
 {
 public:
@@ -54,9 +53,6 @@ class WsTools
     static bool stringToEndPoint(const std::string& peer, NodeIPEndpoint& _endpoint);
 
     static void close(boost::asio::ip::tcp::socket& skt);
-
-    static std::string moduleName() { return m_moduleName; }
-    static void setModuleName(std::string _moduleName) { m_moduleName = _moduleName; }
 };
 }  // namespace ws
 }  // namespace boostssl