diff --git a/README.md b/README.md index 0a5c3ad7..5037e495 100644 --- a/README.md +++ b/README.md @@ -97,7 +97,8 @@ int main(int, char* []) // syslog example // #ifdef __linux__ - auto syslog_logger = spd::syslog_logger("syslog"); + std::string ident = "my_app"; + auto syslog_logger = spd::syslog_logger("syslog", ident, spd::sinks::syslog::option::PID | spd::sinks::syslog::option::PERROR, "mail" ); syslog_logger->warn("This is warning that will end up in syslog. This is Linux only!"); #endif } diff --git a/example/Makefile b/example/Makefile index 4208786a..42795358 100644 --- a/example/Makefile +++ b/example/Makefile @@ -5,7 +5,7 @@ CXX_DEBUG_FLAGS= -g all: example bench -debug: example-debug bench-debug +debug: example-debug bench-debug example: example.cpp $(CXX) example.cpp -o example $(CXXFLAGS) $(CXX_RELEASE_FLAGS) diff --git a/example/example.cpp b/example/example.cpp index 12d0bca4..a2dcbd1b 100644 --- a/example/example.cpp +++ b/example/example.cpp @@ -86,7 +86,8 @@ int main(int, char* []) // syslog example // #ifdef __linux__ - auto syslog_logger = spd::syslog_logger("syslog"); + std::string ident = "my_ident"; + auto syslog_logger = spd::syslog_logger("syslog", ident, spd::sinks::syslog::option::PID | spd::sinks::syslog::option::PERROR, "mail" ); syslog_logger->warn("This is warning that will end up in syslog. This is Linux only!"); #endif } diff --git a/include/spdlog/common.h b/include/spdlog/common.h index 0569f4b3..c9bf2f00 100644 --- a/include/spdlog/common.h +++ b/include/spdlog/common.h @@ -27,13 +27,42 @@ #include #include +#ifdef __linux__ + +#define SYSLOG_NAMES 1 +#include +#endif + namespace spdlog { + class formatter; + namespace sinks { + class sink; + +#ifdef __linux__ +namespace syslog +{ +namespace option +{ +typedef enum +{ + CONS = LOG_CONS, + NDELAY = LOG_NDELAY, + NOWAIT = LOG_NOWAIT, + ODELAY = LOG_ODELAY, + PERROR = LOG_PERROR, + PID = LOG_PID +} option_enum; } +} +#endif + +} + // Common types across the lib using log_clock = std::chrono::system_clock; diff --git a/include/spdlog/details/format.h b/include/spdlog/details/format.h index b74da1f2..3934f812 100644 --- a/include/spdlog/details/format.h +++ b/include/spdlog/details/format.h @@ -115,8 +115,10 @@ SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. #define FMT_DISALLOW_COPY_AND_ASSIGN(TypeName) \ TypeName(const TypeName&); \ void operator=(const TypeName&) -namespace spdlog { -namespace details { +namespace spdlog +{ +namespace details +{ namespace fmt { @@ -1054,7 +1056,7 @@ public: { default: assert(false); - // Fall through. + // Fall through. case Arg::INT: return FMT_DISPATCH(visit_int(arg.int_value)); case Arg::UINT: @@ -2223,7 +2225,7 @@ void BasicWriter::write_double( // MSVC's printf doesn't support 'F'. type = 'f'; #endif - // Fall through. + // Fall through. case 'E': case 'G': case 'A': @@ -2849,8 +2851,10 @@ fmt::print(format, args...); #define FMT_VARIADIC_W(ReturnType, func, ...) \ FMT_VARIADIC_(wchar_t, ReturnType, func, return func, __VA_ARGS__) -namespace spdlog { -namespace details { +namespace spdlog +{ +namespace details +{ namespace fmt { FMT_VARIADIC(std::string, format, StringRef) diff --git a/include/spdlog/details/pattern_formatter_impl.h b/include/spdlog/details/pattern_formatter_impl.h index 1b31ced4..7c44dd7f 100644 --- a/include/spdlog/details/pattern_formatter_impl.h +++ b/include/spdlog/details/pattern_formatter_impl.h @@ -458,7 +458,7 @@ inline void spdlog::pattern_formatter::handle_flag(char flag) { switch (flag) { - // logger name + // logger name case 'n': _formatters.push_back(std::unique_ptr(new details::name_formatter())); break; diff --git a/include/spdlog/details/spdlog_impl.h b/include/spdlog/details/spdlog_impl.h index 6b559064..c50cef89 100644 --- a/include/spdlog/details/spdlog_impl.h +++ b/include/spdlog/details/spdlog_impl.h @@ -87,11 +87,10 @@ inline std::shared_ptr spdlog::stderr_logger_st(const std::strin #ifdef __linux__ // Create syslog logger -inline std::shared_ptr spdlog::syslog_logger(const std::string& logger_name) +inline std::shared_ptr spdlog::syslog_logger(const std::string& logger_name, const std::string& ident, int option, const std::string &facility) { - return create(logger_name); + return create(logger_name, ident, option, facility); } - #endif diff --git a/include/spdlog/sinks/syslog_sink.h b/include/spdlog/sinks/syslog_sink.h index 2aee3f7c..d756b9fd 100644 --- a/include/spdlog/sinks/syslog_sink.h +++ b/include/spdlog/sinks/syslog_sink.h @@ -27,7 +27,8 @@ #ifdef __linux__ #include -#include + + #include "./sink.h" #include "../common.h" #include "../details/log_msg.h" @@ -45,8 +46,9 @@ namespace sinks class syslog_sink : public sink { public: - syslog_sink() + syslog_sink(const std::string& ident = "", int option = static_cast(syslog::option::PID), const std::string &facility = "user") { + _priorities[static_cast(level::TRACE)] = LOG_DEBUG; _priorities[static_cast(level::DEBUG)] = LOG_DEBUG; _priorities[static_cast(level::INFO)] = LOG_INFO; @@ -59,18 +61,23 @@ public: _priorities[static_cast(level::ALWAYS)] = LOG_INFO; _priorities[static_cast(level::OFF)] = LOG_INFO; + + ::openlog(ident.c_str(), option, syslog_facility_from_name(facility)); + } + virtual ~syslog_sink() { + ::closelog(); } - virtual ~syslog_sink() = default; syslog_sink(const syslog_sink&) = delete; syslog_sink& operator=(const syslog_sink&) = delete; void log(const details::log_msg &msg) override { - syslog(syslog_prio_from_level(msg), "%s", msg.formatted.str().c_str()); + ::syslog(syslog_prio_from_level(msg), "%s", msg.formatted.str().c_str()); }; protected: + /** * Simply maps spdlog's log level to syslog priority level. */ @@ -81,6 +88,21 @@ protected: private: std::array _priorities; + + inline int syslog_facility_from_name (const std::string & name) + { + if (name.empty()) + return LOG_USER; + + for (int i = 0; facilitynames[i].c_name != NULL; ++i) + { + if (name == facilitynames[i].c_name) + return facilitynames[i].c_val; + } + + return LOG_USER; + + } }; } } diff --git a/include/spdlog/spdlog.h b/include/spdlog/spdlog.h index 15180963..d538a0b4 100644 --- a/include/spdlog/spdlog.h +++ b/include/spdlog/spdlog.h @@ -96,7 +96,8 @@ std::shared_ptr stderr_logger_st(const std::string& logger_name); // Create a syslog logger // #ifdef __linux__ -std::shared_ptr syslog_logger(const std::string& logger_name); + +std::shared_ptr syslog_logger(const std::string& logger_name, const std::string& ident = "", int option = static_cast(sinks::syslog::option::PID), const std::string &facility = "user"); #endif //