From 7ee5b7d24039c5696da5d1a5c90a32f7e1bb38c9 Mon Sep 17 00:00:00 2001
From: Dan Espinosa <30415120+danespinosa@users.noreply.github.com>
Date: Mon, 11 Nov 2024 04:33:32 -0800
Subject: [PATCH] Update IHttpClientLogger.cs and IHttpClientAsyncLogger.cs
 remarks (#109618)

* Update IHttpClientLogger.cs remarks

Updated IHttpClientLogger remarks to provide clearer documentation to a developer implementing this interface

* Update IHttpClientAsyncLogger.cs

Updated remarks to provide the developer implementing this interface clearer documentation

---------

Co-authored-by: Natalia Kondratyeva <knatalia@microsoft.com>
---
 .../src/Logging/IHttpClientAsyncLogger.cs                       | 2 +-
 .../Microsoft.Extensions.Http/src/Logging/IHttpClientLogger.cs  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientAsyncLogger.cs b/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientAsyncLogger.cs
index 307078a00754a..e23abe8db815b 100644
--- a/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientAsyncLogger.cs
+++ b/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientAsyncLogger.cs
@@ -19,7 +19,7 @@ namespace Microsoft.Extensions.Http.Logging
     /// would be called from the corresponding sync code paths.
     /// </para>
     /// <para>
-    /// It is up to the user implementing the interface to decide where (to <see cref="Microsoft.Extensions.Logging.ILogger"/>, or anything else) and what exactly to log.
+    /// It is up to the user implementing the interface to decide whether to use <see cref="Microsoft.Extensions.Logging.ILogger"/>, or anything else, and what exactly to log.
     /// However, the implementation should be mindful about potential adverse side effects of accessing some of the <see cref="HttpRequestMessage"/> or
     /// <see cref="HttpResponseMessage"/> properties, such as reading from a content stream; if possible, such behavior should be avoided.
     /// </para>
diff --git a/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientLogger.cs b/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientLogger.cs
index e9c61ae312959..a0a473ce1cdc1 100644
--- a/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientLogger.cs
+++ b/src/libraries/Microsoft.Extensions.Http/src/Logging/IHttpClientLogger.cs
@@ -11,7 +11,7 @@ namespace Microsoft.Extensions.Http.Logging
     /// </summary>
     /// <remarks>
     /// <para>
-    /// It is up to the user implementing the interface to decide where (to <see cref="Microsoft.Extensions.Logging.ILogger"/>, or anything else) and what exactly to log.
+    /// It is up to the user implementing the interface to decide whether to use <see cref="Microsoft.Extensions.Logging.ILogger"/>, or anything else, and what exactly to log.
     /// However, the implementation should be mindful about potential adverse side effects of accessing some of the <see cref="HttpRequestMessage"/> or
     /// <see cref="HttpResponseMessage"/> properties, such as reading from a content stream; if possible, such behavior should be avoided.
     /// </para>