diff options
author | Czarnowski, Przemyslaw <przemyslaw.hawrylewicz.czarnowski@intel.com> | 2021-05-06 16:22:34 +0300 |
---|---|---|
committer | Czarnowski, Przemyslaw <przemyslaw.hawrylewicz.czarnowski@intel.com> | 2021-05-06 17:18:07 +0300 |
commit | cf73c528b695ffff93523722b2ec9f40e81bd98b (patch) | |
tree | 03e12392cb88f49d0d1f5297d41baac8aa8fc1a1 /src/system.hpp | |
parent | c78bf93d62126d1e777f4f72322290b40f0121f4 (diff) | |
download | virtual-media-cf73c528b695ffff93523722b2ec9f40e81bd98b.tar.xz |
Change log level to Info
Due to security reasons "user/username" has to be removed from the
information that is logged by application.
Sensitive data has been moved to "Debug"" level (lowest one) and default
one has moved one level up to "Info".
Also some important information allowing to catch basic errors has been
upgraded to "Info".
Tested:
Manually, mounting both Legacy mode remote types (HTTPs and CIFS) and
checking if journal for VirtualMedia service does not contain sensitive
information.
Change-Id: Ie6c3a79c94637e3632af76daf957e986b2dd3b6d
Signed-off-by: Czarnowski, Przemyslaw <przemyslaw.hawrylewicz.czarnowski@intel.com>
Diffstat (limited to 'src/system.hpp')
-rw-r--r-- | src/system.hpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/system.hpp b/src/system.hpp index a29b640..cc8bdad 100644 --- a/src/system.hpp +++ b/src/system.hpp @@ -136,11 +136,11 @@ class NBDDevice } if (ioctl(fd, NBD_DISCONNECT) < 0) { - LogMsg(Logger::Debug, "Ioctl failed: \n"); + LogMsg(Logger::Info, "Ioctl failed: \n"); } if (ioctl(fd, NBD_CLEAR_SOCK) < 0) { - LogMsg(Logger::Debug, "Ioctl failed: \n"); + LogMsg(Logger::Info, "Ioctl failed: \n"); } close(fd); } @@ -359,7 +359,7 @@ class Process : public std::enable_shared_from_this<Process> bool spawn(const std::vector<std::string>& args, ExitCb&& onExit) { std::error_code ec; - LogMsg(Logger::Info, "[Process]: Spawning ", app, " (", args, ")"); + LogMsg(Logger::Debug, "[Process]: Spawning ", app, " (", args, ")"); child = boost::process::child( app, boost::process::args(args), (boost::process::std_out & boost::process::std_err) > pipe, ec, @@ -388,7 +388,7 @@ class Process : public std::enable_shared_from_this<Process> while (lineBegin != line.end()) { auto lineEnd = find(lineBegin, line.end(), '\n'); - LogMsg(Logger::Debug, "[Process]: (", name, ") ", + LogMsg(Logger::Info, "[Process]: (", name, ") ", std::string(lineBegin, lineEnd)); if (lineEnd == line.end()) { @@ -400,7 +400,7 @@ class Process : public std::enable_shared_from_this<Process> buffer.consume(x); if (bec) { - LogMsg(Logger::Debug, "[Process]: (", name, + LogMsg(Logger::Info, "[Process]: (", name, ") Loop Error: ", bec); break; } |