Address review comments: Shorten AUDIT log prefixes; change to /var/log

src/common/shared/logging/include/logging/LogUtils.hxx

@@ -15,13 +15,13 @@ enum class LogInstanceId : int {
     Audit = 3,
 };
 
-#define AUDIT_LOGN LOG_(static_cast<int>(LogInstanceId::Audit), plog::none)
-#define AUDIT_LOGF LOG_(static_cast<int>(LogInstanceId::Audit), plog::fatal)
-#define AUDIT_LOGE LOG_(static_cast<int>(LogInstanceId::Audit), plog::error)
-#define AUDIT_LOGW LOG_(static_cast<int>(LogInstanceId::Audit), plog::warning)
-#define AUDIT_LOGI LOG_(static_cast<int>(LogInstanceId::Audit), plog::info)
-#define AUDIT_LOGD LOG_(static_cast<int>(LogInstanceId::Audit), plog::debug)
-#define AUDIT_LOGV LOG_(static_cast<int>(LogInstanceId::Audit), plog::verbose)
+#define AUDITN LOG_(static_cast<int>(LogInstanceId::Audit), plog::none)
+#define AUDITF LOG_(static_cast<int>(LogInstanceId::Audit), plog::fatal)
+#define AUDITE LOG_(static_cast<int>(LogInstanceId::Audit), plog::error)
+#define AUDITW LOG_(static_cast<int>(LogInstanceId::Audit), plog::warning)
+#define AUDITI LOG_(static_cast<int>(LogInstanceId::Audit), plog::info)
+#define AUDITD LOG_(static_cast<int>(LogInstanceId::Audit), plog::debug)
+#define AUDITV LOG_(static_cast<int>(LogInstanceId::Audit), plog::verbose)
 
 namespace logging
 {

src/linux/server/Main.cxx

@@ -83,12 +83,12 @@ main(int argc, char *argv[])
 
     // Create file and console log appenders.
     static plog::ColorConsoleAppender<plog::MessageOnlyFormatter> colorConsoleAppender{};
-    std::string logFilePath = "/usr/local/";
+    std::string logFilePath = "/var/log/";
     logFilePath += logging::GetLogName("server");
     static plog::RollingFileAppender<plog::TxtFormatter> rollingFileAppender(logFilePath.c_str());
 
     // Create the audit log file appender.
-    std::string auditLogFilePath = "/usr/local/";
+    std::string auditLogFilePath = "/var/log/";
     auditLogFilePath += logging::GetLogName("audit");
     static plog::RollingFileAppender<plog::TxtFormatter> auditLogRollingFileAppender(auditLogFilePath.c_str());
 
@@ -106,7 +106,7 @@ main(int argc, char *argv[])
     plog::init<std::to_underlying(LogInstanceId::Audit)>(logSeverity, &auditLogRollingFileAppender);
 
     LOGN << std::format("Netremote server starting (log level={})", magic_enum::enum_name(logSeverity));
-    AUDIT_LOGN << std::format("Netremote server starting (log level={})", magic_enum::enum_name(logSeverity));
+    AUDITN << std::format("Netremote server starting (log level={})", magic_enum::enum_name(logSeverity));
 
     // Create an access point manager and discovery agent.
     auto accessPointManager = AccessPointManager::Create();