Skip to content

Commit

Permalink
Fix incorrect usage of oxen-logging syslog
Browse files Browse the repository at this point in the history
Previously oxen-logging was erroneously hard-coded to use the target
"lokinet" for system logs.  Obviously this is wrong for anything else
which uses oxen-logging and the system log.  This changes our call to
add_sink to pass "lokinet" as the target rather than the config
filename, and updates oxen-logging to use that argument correctly.
  • Loading branch information
tewinget committed Jan 17, 2023
1 parent ae9fd9a commit fbfd70a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion external/oxen-logging
2 changes: 1 addition & 1 deletion llarp/router/router.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@ namespace llarp
if (log::get_level_default() != log::Level::off)
log::reset_level(conf.logging.m_logLevel);
log::clear_sinks();
log::add_sink(log_type, conf.logging.m_logFile);
log::add_sink(log_type, log_type == log::Type::System ? "lokinet" : conf.logging.m_logFile);

enableRPCServer = conf.api.m_enableRPCServer;

Expand Down

0 comments on commit fbfd70a

Please sign in to comment.