Added Activity IsAllDataRequested whether span should be sampled / recorded (#140)

Author: snakefoot

README.md

@@ -73,6 +73,7 @@ Example of `NLog.config` file that outputs span-details together with LogEvent b
 - TraceStateString : W3C 'tracestate' header as a string
 - Status : Status code of the current activity (`Ok` or `Error`). For int-value combine with `format="d"` 
 - StatusDescription : Status description of the current activity
+- IsAllDataRequested : Activity span has been marked to be sampled / recorded (`0` or `1`)
 
 **Formatting**
 - Format: Format for rendering the property.

src/NLog.DiagnosticSource/LayoutRenderers/ActivityTraceLayoutRenderer.cs

@@ -190,6 +190,7 @@ private string GetValue(System.Diagnostics.Activity activity)
                 case ActivityTraceProperty.TraceStateString: return activity.TraceStateString;
                 case ActivityTraceProperty.Status: return ConvertToString(activity.Status, Format);
                 case ActivityTraceProperty.StatusDescription: return activity.StatusDescription ?? string.Empty;
+                case ActivityTraceProperty.IsAllDataRequested: return activity.IsAllDataRequested ? "1" : "0";
                 default: return string.Empty;
             }
         }

src/NLog.DiagnosticSource/LayoutRenderers/ActivityTraceProperty.cs

@@ -89,5 +89,9 @@ public enum ActivityTraceProperty
         /// Status description of the current activity
         /// </summary>
         StatusDescription,
+        /// <summary>
+        /// Whether the activity span is sampled / recorded, and should be populated with propagation info and other details.
+        /// </summary>
+        IsAllDataRequested,
     }
 }

src/NLog.DiagnosticSource/NLog.DiagnosticSource.csproj

@@ -3,7 +3,7 @@
   <PropertyGroup>
     <TargetFrameworks>net8.0;net6.0;net462;netstandard2.0</TargetFrameworks>
     <RootNamespace>NLog</RootNamespace>
-    <VersionPrefix>5.2.2</VersionPrefix>
+    <VersionPrefix>5.2.3</VersionPrefix>
     <VersionSuffix></VersionSuffix>
 
     <AssemblyVersion>5.0.0.0</AssemblyVersion>
@@ -25,9 +25,7 @@
     <PackageLicenseExpression>BSD-3-Clause</PackageLicenseExpression>
     <PackageReadmeFile>README.md</PackageReadmeFile>
     <PackageReleaseNotes>
-- Added support for property Status (Ok / Error) (@banko6)
-- Added support for property StatusDescription (@banko6)
-- Translate default TraceId / SpanId to empty string (@snakefoot)
+- Added support for property IsAllDataRequested (@snakefoot)
 
 See https://github.com/NLog/NLog.DiagnosticSource for documentation of ${activity} and DiagnosticListenerTarget
     </PackageReleaseNotes>

test/NLog.DiagnosticSource.Tests/ActivityTraceLayoutRendererTests.cs

@@ -52,6 +52,7 @@ public void TestAllPropertiesWhenActivityNull()
         [InlineData(ActivityTraceProperty.ActivityKind, true)]
         [InlineData(ActivityTraceProperty.Status, true)]
         [InlineData(ActivityTraceProperty.StatusDescription, true)]
+        [InlineData(ActivityTraceProperty.IsAllDataRequested, false)]
         public void TestAllPropertiesWhenActivityEmpty(ActivityTraceProperty property, bool empty)
         {
             bool orgThrowExceptions = LogManager.ThrowExceptions;
@@ -106,6 +107,7 @@ public void TestAllPropertiesWhenActivityEmpty(ActivityTraceProperty property, b
         [InlineData(ActivityTraceProperty.Status, "D", "0")]
         [InlineData(ActivityTraceProperty.Status, "X", "00000000")]
         [InlineData(ActivityTraceProperty.StatusDescription, null, "")]
+        [InlineData(ActivityTraceProperty.IsAllDataRequested, null, "1")]
         public void TestAllPropertiesWhenActivityRunning(ActivityTraceProperty property, string format, string output)
         {
             bool orgThrowExceptions = LogManager.ThrowExceptions;