Merge pull request #2 from FizzBuzz791/fix/base-url

fix: set base url in constructor
FizzBuzz791 · Aug 5, 2024 · fbbaa98 · fbbaa98
2 parents 337df8e + ae65dfa
commit fbbaa98
Showing 1 changed file with 8 additions and 3 deletions.
diff --git a/src/HealthCheckService.cs b/src/HealthCheckService.cs
@@ -2,12 +2,18 @@
 
 namespace NHealthCheck;
 
-public class HealthCheckService(HttpClient httpClient) : IHealthCheckService
+public class HealthCheckService : IHealthCheckService
 {
-    private readonly HttpClient HttpClient = httpClient;
+    private readonly HttpClient HttpClient;
 
     private string BaseUrl { get; set; } = "https://hc-ping.com";
 
+    public HealthCheckService(HttpClient httpClient)
+    {
+        HttpClient = httpClient;
+        HttpClient.BaseAddress = new Uri(BaseUrl);
+    }
+
     public async Task<HttpResponseMessage> SuccessAsync(Guid uuid)
     {
         var url = $"{BaseUrl}/{uuid}";
@@ -24,7 +30,6 @@ private async Task<HttpResponseMessage> CallEndpoint(string url)
     {
         try
         {
-            HttpClient.BaseAddress = new Uri(BaseUrl);
             return await HttpClient.GetAsync(url);
         }
         catch (Exception ex)