From 15994393deb646f69c3f20394c7bad47f3f26afe Mon Sep 17 00:00:00 2001
From: MrSmoke <709976+MrSmoke@users.noreply.github.com>
Date: Mon, 20 Oct 2025 17:22:30 +1100
Subject: [PATCH 1/2] Add DefaultUserAgent option

Sets the default user agent for the client
---
 .../src/Http/CoreRestClientOptions.cs         |  1 +
 src/RestClient/RestClient/src/RestClient.cs   | 27 +++++++++++++------
 2 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/src/RestClient/RestClient/src/Http/CoreRestClientOptions.cs b/src/RestClient/RestClient/src/Http/CoreRestClientOptions.cs
index a5e648a..e7fbbad 100644
--- a/src/RestClient/RestClient/src/Http/CoreRestClientOptions.cs
+++ b/src/RestClient/RestClient/src/Http/CoreRestClientOptions.cs
@@ -6,5 +6,6 @@ public class CoreRestClientOptions
     {
         public CompressionMethod CompressionMethod { get; set; } = CompressionMethod.None;
         public IAuthenticator? Authenticator { get; set; } = null;
+        public string? DefaultUserAgent { get; set; }
     }
 }
diff --git a/src/RestClient/RestClient/src/RestClient.cs b/src/RestClient/RestClient/src/RestClient.cs
index 0d356f7..bc1ceb6 100644
--- a/src/RestClient/RestClient/src/RestClient.cs
+++ b/src/RestClient/RestClient/src/RestClient.cs
@@ -1,7 +1,8 @@
-namespace ClickView.Extensions.RestClient
+namespace ClickView.Extensions.RestClient
 {
     using System;
     using System.Net.Http;
+    using System.Net.Http.Headers;
     using System.Threading;
     using System.Threading.Tasks;
     using Exceptions;
@@ -15,6 +16,7 @@ public class RestClient
         private readonly Uri? _baseAddress;
         private readonly HttpClient _httpClient;
         private readonly CoreRestClientOptions _options;
+        private readonly ProductInfoHeaderValue? _userAgent;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="RestClient" /> class
@@ -26,8 +28,9 @@ public RestClient(Uri baseAddress, RestClientOptions? options = null)
             _baseAddress = baseAddress ?? throw new ArgumentNullException(nameof(baseAddress));
 
             var o = options ?? RestClientOptions.Default;
-            _options = o;
 
+            _options = o;
+            _userAgent = GetDefaultUserAgent(_options.DefaultUserAgent);
             _httpClient = CreateClient(o);
         }
 
@@ -40,6 +43,8 @@ public RestClient(HttpClient httpClient, CoreRestClientOptions? options = null)
         {
             _httpClient = httpClient ?? throw new ArgumentNullException(nameof(httpClient));
             _options = options ?? RestClientOptions.Default;
+            _userAgent = GetDefaultUserAgent(_options.DefaultUserAgent);
+
             _baseAddress = null;
         }
 
@@ -71,8 +76,6 @@ public async Task<TResponse> ExecuteAsync<TResponse>(BaseRestClientRequest<TResp
             if (request == null)
                 throw new ArgumentNullException(nameof(request));
 
-            token.ThrowIfCancellationRequested();
-
             if (_options.Authenticator != null)
                 await _options.Authenticator.AuthenticateAsync(request, token).ConfigureAwait(false);
 
@@ -93,7 +96,6 @@ public async Task<TResponse> ExecuteAsync<TResponse>(BaseRestClientRequest<TResp
                 {
                     // only do the following logic if the _httpClient doesn't have a base address already
                     // we should remove this if we update the ctor to always have a baseAddress
-
                     if (_httpClient.BaseAddress == null)
                     {
                         throw new InvalidOperationException(
@@ -105,10 +107,14 @@ public async Task<TResponse> ExecuteAsync<TResponse>(BaseRestClientRequest<TResp
             }
 
             using var httpRequest = new HttpRequestMessage(request.Method, requestUri);
+
+            // Add user agent if supplied
+            if (_userAgent is not null)
+                httpRequest.Headers.UserAgent.Add(_userAgent);
+
+            // Add all the headers from the request and validate them.
             foreach (var h in request.Headers)
-            {
-                httpRequest.Headers.TryAddWithoutValidation(h.Key, h.Value);
-            }
+                httpRequest.Headers.Add(h.Key, h.Value);
 
             httpRequest.Content = GetContent(request);
 
@@ -153,5 +159,10 @@ private static HttpClient CreateClient(RestClientOptions options)
                 Timeout = options.Timeout
             };
         }
+
+        private static ProductInfoHeaderValue? GetDefaultUserAgent(string? userAgent)
+        {
+            return string.IsNullOrEmpty(userAgent) ? null : ProductInfoHeaderValue.Parse(userAgent);
+        }
     }
 }

From 33de8925a95c895ae1d4bc14001d6a8b29139f6b Mon Sep 17 00:00:00 2001
From: MrSmoke <709976+MrSmoke@users.noreply.github.com>
Date: Mon, 20 Oct 2025 17:29:56 +1100
Subject: [PATCH 2/2] Simplify GetDefaultUserAgent check

---
 src/RestClient/RestClient/src/RestClient.cs | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/RestClient/RestClient/src/RestClient.cs b/src/RestClient/RestClient/src/RestClient.cs
index bc1ceb6..af0514b 100644
--- a/src/RestClient/RestClient/src/RestClient.cs
+++ b/src/RestClient/RestClient/src/RestClient.cs
@@ -162,7 +162,10 @@ private static HttpClient CreateClient(RestClientOptions options)
 
         private static ProductInfoHeaderValue? GetDefaultUserAgent(string? userAgent)
         {
-            return string.IsNullOrEmpty(userAgent) ? null : ProductInfoHeaderValue.Parse(userAgent);
+            if (userAgent is null)
+                return null;
+
+            return ProductInfoHeaderValue.Parse(userAgent);
         }
     }
 }