Merge pull request #26 from cnblogs/fix-authentication-regression

fix: authentication regression

Author: ikesnowy

src/Cnblogs.DashScope.Core/DashScopeClient.cs

@@ -1,4 +1,5 @@
-using Cnblogs.DashScope.Core.Internals;
+using System.Net.Http.Headers;
+using Cnblogs.DashScope.Core.Internals;
 
 namespace Cnblogs.DashScope.Core;
 
@@ -33,6 +34,8 @@ private static HttpClient GetConfiguredClient(string apiKey, TimeSpan? timeout)
                 BaseAddress = new Uri(DashScopeDefaults.DashScopeApiBaseAddress),
                 Timeout = timeout ?? TimeSpan.FromMinutes(2)
             };
+
+            client.DefaultRequestHeaders.Authorization = new AuthenticationHeaderValue("Bearer", apiKey);
             ClientPools.Add(GetCacheKey(), client);
         }
 

test/Cnblogs.DashScope.Sdk.UnitTests/DashScopeClientTests.cs

@@ -1,4 +1,5 @@
-using System.Reflection;
+using System.Net.Http.Headers;
+using System.Reflection;
 using Cnblogs.DashScope.Core;
 using FluentAssertions;
 
@@ -59,6 +60,21 @@ public void DashScopeClient_Constructor_CacheableParams(
         value.Should().Be(value2);
     }
 
+    [Fact]
+    public void DashScopeClient_Constructor_WithApiKeyHeader()
+    {
+        // Arrange
+        const string apiKey = "key";
+        var client = new DashScopeClient(apiKey);
+
+        // Act
+        var value = HttpClientAccessor.GetValue(client) as HttpClient;
+
+        // Assert
+        value?.DefaultRequestHeaders.Authorization?.Should()
+            .BeEquivalentTo(new AuthenticationHeaderValue("Bearer", apiKey));
+    }
+
     public static TheoryData<string, string, TimeSpan?, TimeSpan?> ParamsShouldNotCache
         => new()
         {