[api] Mark ActivityExtensions.RecordException obsolete

src/OpenTelemetry.Api/CHANGELOG.md

@@ -6,6 +6,12 @@ Notes](../../RELEASENOTES.md).
 
 ## Unreleased
 
+* Obsoleted the `ActivityExtensions.RecordException`. Users should migrate to the
+  `System.Diagnostics.Activity.AddException`
+  [Activity.AddException](https://learn.microsoft.com/dotnet/api/system.diagnostics.activity.addexception)
+  API for adding exception.
+  ([#5841](https://github.com/open-telemetry/opentelemetry-dotnet/pull/5841))
+
 * **Breaking change:** CompositeTextMapPropagator.Fields now returns a
   unioned set of fields from all combined propagators. Previously this always
   returned an empty set.

src/OpenTelemetry.Api/Trace/ActivityExtensions.cs

@@ -96,7 +96,9 @@ public static Status GetStatus(this Activity? activity)
     /// <param name="ex">Exception to be recorded.</param>
     /// <remarks> The exception is recorded as per <a href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/exceptions.md">specification</a>.
     /// "exception.stacktrace" is represented using the value of <a href="https://learn.microsoft.com/dotnet/api/system.exception.tostring">Exception.ToString</a>.
+    /// <para>This method is obsolete. Please use <see cref="Activity.AddException"/> instead. The end result will be the same when using this alternative.</para>
     /// </remarks>
+    [Obsolete("Call Activity.AddException instead this method will be removed in a future version.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static void RecordException(this Activity? activity, Exception? ex)
         => RecordException(activity, ex, default);
@@ -109,7 +111,9 @@ public static void RecordException(this Activity? activity, Exception? ex)
     /// <param name="tags">Additional tags to record on the event.</param>
     /// <remarks> The exception is recorded as per <a href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/exceptions.md">specification</a>.
     /// "exception.stacktrace" is represented using the value of <a href="https://learn.microsoft.com/dotnet/api/system.exception.tostring">Exception.ToString</a>.
+    /// <para>This method is obsolete. Please use <see cref="Activity.AddException"/> instead. The end result will be the same when using this alternative.</para>
     /// </remarks>
+    [Obsolete("Call Activity.AddException instead this method will be removed in a future version.")]
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
     public static void RecordException(this Activity? activity, Exception? ex, in TagList tags)
     {
@@ -118,22 +122,6 @@ public static void RecordException(this Activity? activity, Exception? ex, in Ta
             return;
         }
 
-        var tagsCollection = new ActivityTagsCollection
-        {
-            { SemanticConventions.AttributeExceptionType, ex.GetType().FullName },
-            { SemanticConventions.AttributeExceptionStacktrace, ex.ToInvariantString() },
-        };
-
-        if (!string.IsNullOrWhiteSpace(ex.Message))
-        {
-            tagsCollection.Add(SemanticConventions.AttributeExceptionMessage, ex.Message);
-        }
-
-        foreach (var tag in tags)
-        {
-            tagsCollection[tag.Key] = tag.Value;
-        }
-
-        activity.AddEvent(new ActivityEvent(SemanticConventions.AttributeExceptionEventName, default, tagsCollection));
+        activity.AddException(ex, tags);
     }
 }