在spdlog自定义Sink中处理日志死锁问题

问题背景

在使用spdlog日志库开发自定义TCP Sink时，开发者遇到了一个潜在的死锁问题。当TCP连接断开时，Sink内部需要记录错误日志，但如果直接使用同一个logger实例，会导致递归调用和死锁。

问题分析

在自定义Sink的sink_it_方法中，当网络连接出现问题时（如"Connection reset by peer"），开发者希望记录错误日志。但如果直接使用同一个logger实例，会导致：

1. 原始日志消息进入logger
2. logger将消息分发给所有Sink，包括TCP Sink
3. TCP Sink处理时遇到错误，尝试记录错误日志
4. 错误日志再次进入同一个logger，形成递归调用
5. 最终导致死锁

解决方案

针对这个问题，开发者提出了一个合理的解决方案：

1. 创建两个独立的logger实例
   • 主logger：包含控制台和文件Sink
   • TCP专用logger：仅包含TCP Sink
2. 在TCP Sink内部需要记录错误时，使用主logger而非TCP专用logger

这种方案有效避免了递归调用，因为：

• TCP Sink内部的错误日志不会再次进入TCP Sink
• 错误日志只会被主logger的Sink处理

实现建议

在实际实现时，可以考虑以下优化：

1. 将主logger作为TCP Sink的构造参数传入
2. 在TCP Sink内部使用主logger记录错误
3. 确保两个logger使用不同的互斥锁

示例代码结构：

// 创建主logger
auto main_logger = std::make_shared<spdlog::logger>("main");
main_logger->sinks().push_back(console_sink);
main_logger->sinks().push_back(file_sink);

// 创建TCP Sink，传入主logger
auto tcp_sink = std::make_shared<TcpSink>(server, main_logger);

// 创建TCP专用logger
auto tcp_logger = std::make_shared<spdlog::logger>("tcp");
tcp_logger->sinks().push_back(tcp_sink);

其他注意事项

1. 线程安全：确保不同logger使用不同的互斥锁
2. 性能考虑：TCP Sink内部的错误日志应尽量简洁
3. 错误处理：考虑添加重连机制而非频繁记录错误
4. 日志级别：可以适当降低TCP Sink内部错误日志的级别

结论

在spdlog中开发自定义Sink时，特别是在网络相关的Sink中，需要特别注意日志递归调用导致的死锁问题。通过分离logger实例的方式，可以有效地解决这个问题，同时保持日志系统的完整性和可用性。