From bbe23f6996ee8d4ee263c3803953ba7b89221331 Mon Sep 17 00:00:00 2001
From: Adam Hathcock <adamhathcock@users.noreply.github.com>
Date: Mon, 16 Sep 2024 11:45:04 +0100
Subject: [PATCH] Update Http Send trace (#118)

* Update Http Send trace

* format
---
 .../Helpers/SpeckleHttpClientHandler.cs       | 29 ++++++-------------
 1 file changed, 9 insertions(+), 20 deletions(-)

diff --git a/src/Speckle.Sdk/Helpers/SpeckleHttpClientHandler.cs b/src/Speckle.Sdk/Helpers/SpeckleHttpClientHandler.cs
index 9669c091..49744d19 100644
--- a/src/Speckle.Sdk/Helpers/SpeckleHttpClientHandler.cs
+++ b/src/Speckle.Sdk/Helpers/SpeckleHttpClientHandler.cs
@@ -1,4 +1,4 @@
-using System.Diagnostics;
+using System.Diagnostics;
 using Polly;
 using Speckle.Sdk.Common;
 using Speckle.Sdk.Logging;
@@ -24,16 +24,10 @@ CancellationToken cancellationToken
   {
     // this is a preliminary client server correlation implementation
     // refactor this, when we have a better observability stack
-    var sw = Stopwatch.StartNew();
     var context = new Context();
-    using var activity = SpeckleActivityFactory.Start("Http Send");
+    using var activity = SpeckleActivityFactory.Start("Http Request");
     {
-      SpeckleLog.Logger.Debug(
-        "Starting execution of http request to {targetUrl} {correlationId} {traceId}",
-        request.RequestUri,
-        context.CorrelationId,
-        activity?.TraceId
-      );
+      activity?.SetTag("http.method", request.Method);
       activity?.SetTag("http.url", request.RequestUri);
       activity?.SetTag("correlationId", context.CorrelationId);
 
@@ -52,18 +46,13 @@ CancellationToken cancellationToken
         .ConfigureAwait(false);
       context.TryGetValue("retryCount", out var retryCount);
       activity?.SetTag("retryCount", retryCount);
-
-      SpeckleLog.Logger.Information(
-        "Execution of http request to {httpScheme}://{hostUrl}{relativeUrl} {resultStatus} with {httpStatusCode} after {elapsed} seconds and {retryCount} retries. Request correlation ID: {correlationId}",
-        request.RequestUri.Scheme,
-        request.RequestUri.Host,
-        request.RequestUri.PathAndQuery,
-        policyResult.Outcome == OutcomeType.Successful ? "succeeded" : "failed",
-        policyResult.Result?.StatusCode,
-        sw.Elapsed.TotalSeconds,
-        retryCount ?? 0,
-        context.CorrelationId.ToString()
+      activity?.SetStatus(
+        policyResult.Result.IsSuccessStatusCode ? SpeckleActivityStatusCode.Ok : SpeckleActivityStatusCode.Error
       );
+      if (policyResult.FinalException != null)
+      {
+        activity?.RecordException(policyResult.FinalException);
+      }
 
       if (policyResult.Outcome == OutcomeType.Successful)
       {