fix minor comments and adjust logic improvement

ChrstnAnsl · Jun 23, 2024 · 4240536 · 4240536
1 parent 4d42101
commit 4240536
Showing 1 changed file with 5 additions and 8 deletions.
diff --git a/dotnet/src/webdriver/Remote/HttpCommandExecutor.cs b/dotnet/src/webdriver/Remote/HttpCommandExecutor.cs
@@ -414,22 +414,19 @@ public DiagnosticsHttpHandler(HttpMessageHandler messageHandler)
             /// <returns>The http response message content.</returns>
             protected override async Task<HttpResponseMessage> SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
             {
-                if (request.Content != null)
-                {
-                    var requestContent = await request.Content.ReadAsStringAsync().ConfigureAwait(false);
-                    _logger.Trace($">> Body: {requestContent}");
-                }
-
                 var response = await base.SendAsync(request, cancellationToken).ConfigureAwait(false);
 
-                if (!response.IsSuccessStatusCode)
+                if (!response.IsSuccessStatusCode && response.Content != null && request.Content != null)
                 {
+                    var requestContent = await request.Content.ReadAsStringAsync().ConfigureAwait(false);
                     var responseContent = await response.Content.ReadAsStringAsync().ConfigureAwait(false);
-                    _logger.Trace($"<< Body: {responseContent}");
+
+                    _logger.Trace($">> Request Body: {requestContent}\n\n<< Response Body: {responseContent}");
                 }
 
                 return response;
             }
+
         }
     }
 }