kizilsu
kizilsu

Reputation: 431

Net Core NLog.Web "aspnet-request:header" property usage?

Is there any way to log all headers using "aspnet-request:header" property with one parameter? Or should I get headers one by one like "aspnet-request:header=MyHeader" and combine them into one parameter before insert? I have lots of headers and don't want to add them seperately, I need a quick way to log them if its possible.

Upvotes: 1

Views: 831

Answers (1)

Julian
Julian

Reputation: 36790

Currently only one at once header is supported, as it calls

string header = httpRequest.Headers[this.Header]; see source

edit: you could plug it in NLog like this:

using System;
using System.Text;
using Microsoft.AspNetCore.Http;
using NLog.Config;
using NLog.LayoutRenderers;
using NLog.Web.Internal;

namespace NLog.Web.LayoutRenderers
{
    /// <summary>
    /// Render all headers for ASP.NET Core
    /// </summary>
    /// <example>
    /// <code lang="NLog Layout Renderer">
    /// ${aspnet-request-all-headers}
    /// </code>
    /// </example>
    [LayoutRenderer("aspnet-request-all-headers")]
    public class AspNetRequestAllHeadersLayoutRenderer : AspNetLayoutRendererBase
    {


        protected override void DoAppend(StringBuilder builder, LogEventInfo logEvent)
        {
            var httpRequest = HttpContextAccessor.HttpContext.TryGetRequest();
            if (httpRequest == null || httpRequest.Headers == null)
            {
                return;
            }
            foreach(var kpv in httpRequest.Headers)
            {

                if (header != null)
                {
                    builder.Append(kpv.Key);
                    builder.Append(=);
                    builder.Append(kpv.Value);
                }
            }

        }
    }
}

Register it (startup.cs)

    ConfigurationItemFactory.Default.LayoutRenderers
                        .RegisterDefinition("aspnet-request-all-headers", typeof(AspNetRequestAllHeadersLayoutRenderer  ));

See also Extending NLog

usage

 ${aspnet-request-all-headers}

Upvotes: 1

Related Questions