Serilog Enricher中的StackOverflowException

Question

我正在为Serilog编写一个增强器，我想在实践中添加一个可能触发新日志事件的属性：

public void Enrich(LogEvent logEvent, ILogEventPropertyFactory propertyFactory)
{
    ISession session = /* HttpContext.Session */;
    logEvent.AddPropertyIfAbsent(propertyFactory.CreateProperty("ValueFromSession", session.GetString("ValueFromSession")));
    /* other properties ... */
}

我希望能够将我的http会话中的值添加到我的日志中，因此我将LogContext.PushProperties与自定义的LogContext.PushProperties一起使用。 ASP.NET Core会在读取过期会话时记录警告，这会触发我的richher，但每当我尝试从richver中的会话中读取时，它会向我发出关于过期会话的警告，这会触发我的richr的新实例（ad infitum） ）。 这会导致StackOverflowException似乎无法通过try块进行捕获。

在Enrich方法中有没有办法禁用日志记录？ 或者我可以以某种方式检测这种递归（除了解析堆栈跟踪）？

Answer 1

我的自定义ILogger实现（不使用Serilog）遇到了这个问题。 它会在访问Session对象时发生。 我设法通过在启动时将持久密钥的DataProtection添加到我的应用程序来解决这个问题。

services.AddDataProtection()
    .PersistKeysToFileSystem(new DirectoryInfo("keys"));

我没有深入了解实现细节，但是从我在DistributedSession代码中看到的情况，当检测到旧会话密钥时，会记录“访问过期会话”消息。 这里的问题是日志记录发生在isAvailable标志设置为true ，这会导致递归。 我不知道为什么添加DataProtection会避免这个问题，但是DataProtection可能导致_isNewSessionKey始终为true 。

GitHub中建议的另一个解决方法是添加一个中间件来保存要记录到HttpContext.Items的会话项。 然后在记录器中检索这些项目。