Use the same naming rules for the names of logging dimensions

src/Generators/Microsoft.Gen.Logging/Emission/Emitter.Method.cs

@@ -14,6 +14,9 @@ namespace Microsoft.Gen.Logging.Emission;
 
 internal sealed partial class Emitter : EmitterBase
 {
+    private const string PropertySeparator = ".";
+    private const string NullablePropertySeparator = "?.";
+
 #pragma warning disable CA1505 // Avoid unmaintainable code
     private void GenLogMethod(LoggingMethod lm)
 #pragma warning restore CA1505
@@ -316,8 +319,8 @@ void GenPropertyLoads(LoggingMethod lm, string stateName, out int numReservedUnc
                         {
                             if (!member.HasDataClassification)
                             {
-                                var propName = PropertyChainToString(propertyChain, member, "_", omitReferenceName: p.OmitReferenceName);
-                                var accessExpression = PropertyChainToString(propertyChain, member, "?.", nonNullSeparator: ".");
+                                var propName = PropertyChainToString(propertyChain, member, PropertySeparator, omitReferenceName: p.OmitReferenceName);
+                                var accessExpression = PropertyChainToString(propertyChain, member, NullablePropertySeparator, nonNullSeparator: PropertySeparator);
 
                                 var ts = ShouldStringifyProperty(member)
                                     ? ConvertPropertyToString(member, accessExpression)
@@ -367,8 +370,8 @@ void GenPropertyLoads(LoggingMethod lm, string stateName, out int numReservedUnc
                         {
                             if (member.HasDataClassification)
                             {
-                                var propName = PropertyChainToString(propertyChain, member, "_", omitReferenceName: p.OmitReferenceName);
-                                var accessExpression = PropertyChainToString(propertyChain, member, "?.", nonNullSeparator: ".");
+                                var propName = PropertyChainToString(propertyChain, member, PropertySeparator, omitReferenceName: p.OmitReferenceName);
+                                var accessExpression = PropertyChainToString(propertyChain, member, NullablePropertySeparator, nonNullSeparator: PropertySeparator);
 
                                 var value = ShouldStringifyProperty(member)
                                     ? ConvertPropertyToString(member, accessExpression)
@@ -391,8 +394,8 @@ void GenPropertyLoads(LoggingMethod lm, string stateName, out int numReservedUnc
                     {
                         if (member.HasDataClassification)
                         {
-                            var propName = PropertyChainToString(propertyChain, member, "_", omitReferenceName: p.OmitReferenceName);
-                            var accessExpression = PropertyChainToString(propertyChain, member, "?.", nonNullSeparator: ".");
+                            var propName = PropertyChainToString(propertyChain, member, PropertySeparator, omitReferenceName: p.OmitReferenceName);
+                            var accessExpression = PropertyChainToString(propertyChain, member, NullablePropertySeparator, nonNullSeparator: PropertySeparator);
 
                             var value = ShouldStringifyProperty(member)
                                 ? ConvertPropertyToString(member, accessExpression)
@@ -418,8 +421,8 @@ void GenPropertyLoads(LoggingMethod lm, string stateName, out int numReservedUnc
                         }
                         else
                         {
-                            var propName = PropertyChainToString(propertyChain, member, "_", omitReferenceName: p.OmitReferenceName);
-                            var accessExpression = PropertyChainToString(propertyChain, member, "?.", nonNullSeparator: ".");
+                            var propName = PropertyChainToString(propertyChain, member, PropertySeparator, omitReferenceName: p.OmitReferenceName);
+                            var accessExpression = PropertyChainToString(propertyChain, member, NullablePropertySeparator, nonNullSeparator: PropertySeparator);
 
                             var ts = ShouldStringifyProperty(member)
                                 ? ConvertPropertyToString(member, accessExpression)
@@ -630,7 +633,7 @@ private string PropertyChainToString(
         string? nonNullSeparator = null,
         bool omitReferenceName = false)
     {
-        bool needAts = nonNullSeparator == ".";
+        bool needAts = nonNullSeparator == PropertySeparator;
         var adjustedNonNullSeparator = nonNullSeparator ?? separator;
         var localStringBuilder = _sbPool.GetStringBuilder();
         try

src/Libraries/Microsoft.AspNetCore.Diagnostics.Middleware/Logging/HttpLoggingTagNames.cs

@@ -19,7 +19,7 @@ public static class HttpLoggingTagNames
     /// <summary>
     /// HTTP Host.
     /// </summary>
-    public const string Host = "Host";
+    public const string Host = "server.address";
 
     /// <summary>
     /// HTTP Method.
@@ -34,12 +34,12 @@ public static class HttpLoggingTagNames
     /// <summary>
     /// HTTP Request Headers prefix.
     /// </summary>
-    public const string RequestHeaderPrefix = "RequestHeader.";
+    public const string RequestHeaderPrefix = "http.request.header.";
 
     /// <summary>
     /// HTTP Response Headers prefix.
     /// </summary>
-    public const string ResponseHeaderPrefix = "ResponseHeader.";
+    public const string ResponseHeaderPrefix = "http.response.header.";
 
     /// <summary>
     /// HTTP Request Body.

src/Libraries/Microsoft.AspNetCore.HeaderParsing/HeaderParsingFeature.cs

@@ -198,12 +198,12 @@ private void CopyTo(Box<T> to)
         }
     }
 
-    [LoggerMessage(LogLevel.Debug, "Can't parse header '{headerName}' due to '{error}'.")]
+    [LoggerMessage(LogLevel.Debug, "Can't parse header '{HeaderName}' due to '{Error}'.")]
     private partial void LogParsingError(string headerName, string error);
 
-    [LoggerMessage(LogLevel.Debug, "Using a default value for header '{headerName}'.")]
+    [LoggerMessage(LogLevel.Debug, "Using a default value for header '{HeaderName}'.")]
     private partial void LogDefaultUsage(string headerName);
 
-    [LoggerMessage(LogLevel.Debug, "Header '{headerName}' not found.")]
+    [LoggerMessage(LogLevel.Debug, "Header '{HeaderName}' not found.")]
     private partial void LogNotFound(string headerName);
 }

src/Libraries/Microsoft.Extensions.Diagnostics.Extra/Enrichment/ApplicationEnricherTags.cs

@@ -14,22 +14,22 @@ public static class ApplicationEnricherTags
     /// <summary>
     /// Application name.
     /// </summary>
-    public const string ApplicationName = "AppName";
+    public const string ApplicationName = "service.name";
 
     /// <summary>
     /// Environment name.
     /// </summary>
-    public const string EnvironmentName = "CloudEnv";
+    public const string EnvironmentName = "deployment.environment";
 
     /// <summary>
     /// Deployment ring.
     /// </summary>
-    public const string DeploymentRing = "CloudDeploymentRing";
+    public const string DeploymentRing = "DeploymentRing";
 
     /// <summary>
     /// Build version.
     /// </summary>
-    public const string BuildVersion = "CloudRoleVer";
+    public const string BuildVersion = "service.version";
 
     /// <summary>
     /// Gets a list of all dimension names.

src/Libraries/Microsoft.Extensions.Diagnostics.Extra/Enrichment/ProcessEnricherTagNames.cs

@@ -14,12 +14,12 @@ public static class ProcessEnricherTagNames
     /// <summary>
     /// Process ID.
     /// </summary>
-    public const string ProcessId = "pid";
+    public const string ProcessId = "process.pid";
 
     /// <summary>
     /// Thread ID.
     /// </summary>
-    public const string ThreadId = "tid";
+    public const string ThreadId = "thread.id";
 
     /// <summary>
     /// Gets a list of all dimension names.

src/Libraries/Microsoft.Extensions.Diagnostics.ExtraAbstractions/Logging/LoggerMessageState.TagCollector.cs

@@ -7,7 +7,7 @@ namespace Microsoft.Extensions.Logging;
 
 public partial class LoggerMessageState : ITagCollector
 {
-    private const char Separator = '_';
+    private const char Separator = '.';
 
     /// <inheritdoc />
     void ITagCollector.Add(string tagName, object? tagValue)

src/Libraries/Microsoft.Extensions.Diagnostics.HealthChecks.Common/Log.cs

@@ -8,13 +8,13 @@ namespace Microsoft.Extensions.Diagnostics.HealthChecks;
 
 internal static partial class Log
 {
-    [LoggerMessage(0, LogLevel.Warning, "Process reporting unhealthy: {status}. Health check entries are {entries}")]
+    [LoggerMessage(0, LogLevel.Warning, "Process reporting unhealthy: {Status}. Health check entries are {Entries}")]
     public static partial void Unhealthy(
         ILogger logger,
         HealthStatus status,
         StringBuilder entries);
 
-    [LoggerMessage(1, LogLevel.Debug, "Process reporting healthy: {status}.")]
+    [LoggerMessage(1, LogLevel.Debug, "Process reporting healthy: {Status}.")]
     public static partial void Healthy(
         ILogger logger,
         HealthStatus status);

src/Libraries/Microsoft.Extensions.Diagnostics.ResourceMonitoring/Log.cs

@@ -14,6 +14,6 @@ internal static partial class Log
     [LoggerMessage(1, LogLevel.Error, "Unable to gather utilization statistics.")]
     public static partial void HandledGatherStatisticsException(ILogger logger, Exception e);
 
-    [LoggerMessage(2, LogLevel.Error, "Publisher `{publisher}` was unable to publish utilization statistics.")]
+    [LoggerMessage(2, LogLevel.Error, "Publisher `{Publisher}` was unable to publish utilization statistics.")]
     public static partial void HandlePublishUtilizationException(ILogger logger, Exception e, string publisher);
 }

src/Libraries/Microsoft.Extensions.Hosting.Testing/HostTerminatorService.cs

@@ -60,7 +60,7 @@ protected override async Task ExecuteAsync(CancellationToken stoppingToken)
         _host.Dispose();
     }
 
-    [LoggerMessage(0, LogLevel.Warning, "FakeHostOptions.TimeToLive set to {timeToLive} is up, disposing the host.")]
+    [LoggerMessage(0, LogLevel.Warning, "FakeHostOptions.TimeToLive set to {TimeToLive} is up, disposing the host.")]
     private partial void LogTimeToLiveUp(TimeSpan timeToLive);
 
     [LoggerMessage(1, LogLevel.Information, "Debugger is attached. The host won't be automatically disposed.")]

src/Libraries/Microsoft.Extensions.Http.Diagnostics/Latency/Internal/HttpClientLatencyLogEnricher.cs

@@ -50,7 +50,7 @@ public void Enrich(IEnrichmentTagCollector collector, HttpRequestMessage request
                 AppendCheckpoints(lc, stringBuilder);
             }
 
-            collector.Add("latencyInfo", stringBuilder.ToString());
+            collector.Add("LatencyInfo", stringBuilder.ToString());
 
             _builderPool.Return(stringBuilder);
         }

src/Libraries/Microsoft.Extensions.Http.Diagnostics/Logging/HttpClientLoggingTagNames.cs

@@ -14,47 +14,47 @@ public static class HttpClientLoggingTagNames
     /// <summary>
     /// HTTP Request duration.
     /// </summary>
-    public const string Duration = "duration";
+    public const string Duration = "Duration";
 
     /// <summary>
     /// HTTP Host.
     /// </summary>
-    public const string Host = "httpHost";
+    public const string Host = "server.address";
 
     /// <summary>
     /// HTTP Method.
     /// </summary>
-    public const string Method = "httpMethod";
+    public const string Method = "http.request.method";
 
     /// <summary>
     /// HTTP Path.
     /// </summary>
-    public const string Path = "httpPath";
+    public const string Path = "url.path";
 
     /// <summary>
     /// HTTP Request Body.
     /// </summary>
-    public const string RequestBody = "httpRequestBody";
+    public const string RequestBody = "RequestBody";
 
     /// <summary>
     /// HTTP Response Body.
     /// </summary>
-    public const string ResponseBody = "httpResponseBody";
+    public const string ResponseBody = "ResponseBody";
 
     /// <summary>
     /// HTTP Request Headers prefix.
     /// </summary>
-    public const string RequestHeaderPrefix = "httpRequestHeader_";
+    public const string RequestHeaderPrefix = "http.request.header.";
 
     /// <summary>
     /// HTTP Response Headers prefix.
     /// </summary>
-    public const string ResponseHeaderPrefix = "httpResponseHeader_";
+    public const string ResponseHeaderPrefix = "http.response.header.";
 
     /// <summary>
     /// HTTP Status Code.
     /// </summary>
-    public const string StatusCode = "httpStatusCode";
+    public const string StatusCode = "http.response.status_code";
 
     /// <summary>
     /// Gets a list of all tag names.