Update src/Hosting/Hosting/src/Internal/HostingApplicationDiagnostics.cshalter73/delete-old-comments

1 files changed, 4 insertions, 0 deletions

diff --git a/src/Hosting/Hosting/src/Internal/HostingApplicationDiagnostics.cs b/src/Hosting/Hosting/src/Internal/HostingApplicationDiagnostics.cs
index 6495fa9c94..4c51043c9e 100644
--- a/src/Hosting/Hosting/src/Internal/HostingApplicationDiagnostics.cs
+++ b/src/Hosting/Hosting/src/Internal/HostingApplicationDiagnostics.cs
@@ -328,6 +328,10 @@ internal sealed class HostingApplicationDiagnostics
         {
             if (ActivityContext.TryParse(requestId, traceState, isRemote: true, out ActivityContext context))
             {
+                // The requestId used the W3C ID format. Unfortunately, the ActivitySource.CreateActivity overload that
+                // takes a string parentId never sets HasRemoteParent to true. We work around that by calling the
+                // ActivityContext overload instead which sets HasRemoteParent to parentContext.IsRemote.
+                // https://github.com/dotnet/aspnetcore/pull/41568#discussion_r868733305
                 activity = _activitySource.CreateActivity(ActivityName, ActivityKind.Server, context);
             }
             else