summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGökay Şatır <gokaysatir@collabora.com>2020-11-30 11:58:57 +0300
committerAshod Nakashian <Ashod@users.noreply.github.com>2020-12-07 17:30:51 -0500
commitcdd10066df780c9a882ba20a09be09716a8d0843 (patch)
tree2bada755aceeef7f68c065c1def58c4d02f4e881
parentAdmin console: Cosmetic changes and documentation. (diff)
downloadonline-cdd10066df780c9a882ba20a09be09716a8d0843.tar.gz
online-cdd10066df780c9a882ba20a09be09716a8d0843.zip
AdminConsole: Additional changes based on reviews.
Change-Id: I277d9179a84dc34aae9770a07a3a72d35a24a0a6 Signed-off-by: Gökay Şatır <gokaysatir@collabora.com>
-rw-r--r--kit/ForKit.cpp2
-rw-r--r--wsd/Admin.cpp6
-rw-r--r--wsd/DocumentBroker.cpp1
3 files changed, 4 insertions, 5 deletions
diff --git a/kit/ForKit.cpp b/kit/ForKit.cpp
index a16a44adef..35ef20f8e3 100644
--- a/kit/ForKit.cpp
+++ b/kit/ForKit.cpp
@@ -155,7 +155,7 @@ protected:
else if (tokens.size() == 2 && tokens.equals(0, "setloglevel"))
{
// Set environment variable so that new children will also set their log levels accordingly.
- setenv("LOOL_LOGLEVEL", tokens[1].c_str(), true);
+ setenv("LOOL_LOGLEVEL", tokens[1].c_str(), 1);
Log::logger().setLevel(tokens[1]);
}
else if (tokens.size() == 3 && tokens.equals(0, "setconfig"))
diff --git a/wsd/Admin.cpp b/wsd/Admin.cpp
index cfca8c6ed0..238d00bfc7 100644
--- a/wsd/Admin.cpp
+++ b/wsd/Admin.cpp
@@ -606,7 +606,7 @@ std::string Admin::getChannelLogLevels()
unsigned int wsdLogLevel = Log::logger().get("wsd").getLevel();
std::string result = "wsd=" + levelList[wsdLogLevel];
- result += " kits=" + (_forkitLogLevel.empty() != true ? _forkitLogLevel: levelList[wsdLogLevel]);
+ result += " kit=" + (_forkitLogLevel.empty() != true ? _forkitLogLevel: levelList[wsdLogLevel]);
return result;
}
@@ -620,11 +620,11 @@ void Admin::setChannelLogLevel(const std::string& _channelName, std::string leve
Log::logger().names(nameList);
if (std::find(std::begin(levelList), std::end(levelList), level) == std::end(levelList))
- level = "trace";
+ level = "debug";
if (_channelName == "wsd")
Log::logger().get("wsd").setLevel(level);
- else if (_channelName == "kits")
+ else if (_channelName == "kit")
{
LOOLWSD::setLogLevelsOfKits(level); // For current kits.
LOOLWSD::sendMessageToForKit("setloglevel " + level); // For forkit and future kits.
diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp
index 8450642378..867dc791bb 100644
--- a/wsd/DocumentBroker.cpp
+++ b/wsd/DocumentBroker.cpp
@@ -1723,7 +1723,6 @@ void DocumentBroker::alertAllUsers(const std::string& msg)
void DocumentBroker::setKitLogLevel(const std::string& level)
{
assertCorrectThread();
- std::unique_lock<std::mutex> lock(_mutex);
_childProcess->sendTextFrame("setloglevel " + level);
}