fix remote logger remove ServerConfig::Context

TarsCloud · Jan 12, 2024 · e4c873f · e4c873f
1 parent 4fdea5f
commit e4c873f
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/servant/libservant/RemoteLogger.cpp b/servant/libservant/RemoteLogger.cpp
@@ -114,7 +114,9 @@ void RollWriteT::setDyeingLogInfo(const string &sApp, const string &sServer, con
 
 LocalRollLogger::~LocalRollLogger()
 {
-	for(auto e : _logger_ex)
+    terminate();
+
+	for(const auto& e : _logger_ex)
 	{
 		delete e.second;
 	}
@@ -518,7 +520,7 @@ void TimeWriteT::operator()(ostream &of, const deque<pair<size_t, string> > &buf
     {
         try
         {
-            _logPrx->logger(DYEING_DIR, DYEING_FILE, "day", "%Y%m%d", vDyeingLog, ServerConfig::Context);
+            _logPrx->logger(DYEING_DIR, DYEING_FILE, "day", "%Y%m%d", vDyeingLog, _logPrx->tars_communicator()->getClientConfig().Context);
         }
         catch(exception &ex)
         {

diff --git a/servant/libservant/ServantHandle.cpp b/servant/libservant/ServantHandle.cpp
@@ -581,7 +581,7 @@ void ServantHandle::handleTarsProtocol(const CurrentPtr &current)
 
     int ret = TARSSERVERUNKNOWNERR;
 
-    string sResultDesc = "";
+    string sResultDesc;
 
     ResponsePacket response;
 

diff --git a/servant/libservant/ServantProxy.cpp b/servant/libservant/ServantProxy.cpp
@@ -814,12 +814,14 @@ uint32_t ServantProxy::tars_gen_requestid()
 
 void ServantProxy::tars_set_push_callback(const ServantProxyCallbackPtr & cb)
 {
+    std::lock_guard<std::mutex> lock(_mutex);
 	_pushCallback = cb;
     cb->setServantPrx(this);
 }
 
 ServantProxyCallbackPtr ServantProxy::tars_get_push_callback()
 {
+    std::lock_guard<std::mutex> lock(_mutex);
 	return _pushCallback;
 }
 

diff --git a/servant/protocol b/servant/protocol
+11 −11		framework/RegistryDescriptor.tars
+9 −0		servant/EndpointF.tars