Log CorrelationId with SerilogWeb.Classic's log entries

[serilog-web]

Log CorrelationId with SerilogWeb.Classic's log entries

[tsimbalar]

not 100% sure if I understand correctly 🤔

1. Why do you need to "dispose" of your CorrelationId in the OnUnload ? would "not disposing it" allow it to still exist at the moment where SerilogWeb logs the event ?
2. wouldn't using or copying/adapting the current HttpRequestIdEnricher help ? note that it exposes static accessor in case you need access to the Id of the current request

classic/src/SerilogWeb.Classic/Classic/Enrichers/HttpRequestIdEnricher.cs

Line 56 in f07c97a

public static bool TryGetCurrentHttpRequestId(out Guid requestId)

[mattheys]

Sorry I wasn't clear, I meant about disposing of the IDisposable property from PushProperty("CorrelationId",correlationId) I was using.

I've taken this approach instead since it's a simple single api that returns a document based on url properties and there is no state to manage, I just abandon the session before the page ends. I will take a look at the TryGetCurrentHttpRequestId and see if that helps as well.

        static Dictionary<Guid, IDisposable> logProperties = new Dictionary<Guid, IDisposable>();
        protected void Session_Start(object sender, EventArgs e)
        {
            var guid = Guid.NewGuid();
            Session["CorrelationId"] = guid;
            IDisposable logProperty = Serilog.Context.LogContext.PushProperty("CorrelationId", guid);
            Log.Verbose("Session {CorrelationId} started", guid);
            logProperties.Add(guid, logProperty);
        }
        protected void Session_End(object sender, EventArgs e)
        {
            Guid? guid = Session["CorrelationId"] as Guid?;

            if (!guid.HasValue)
            {
                Log.Verbose("Session with no CorrelationId ended, {CorrelationIdCount} in dictionary", logProperties.Count);
                return;
            }

            Log.Verbose("Session {CorrelationId} ended", guid);
            if (logProperties.TryGetValue(guid.Value, out var log))
            {
                log.Dispose();
                logProperties.Remove(guid.Value);
            }
        }