Merge pull request #103 from pkuehnel/feat/loggingGetOverage

feat(GridService): log requestUri and error reason phrase
pkuehnel · Apr 26, 2022 · 019ff6d · 019ff6d
2 parents d0d29c8 + 12c438f
commit 019ff6d
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/SmartTeslaAmpSetter/Server/Services/GridService.cs b/SmartTeslaAmpSetter/Server/Services/GridService.cs
@@ -21,10 +21,17 @@ public async Task<int> GetCurrentOverage()
         _logger.LogTrace("{method}()", nameof(GetCurrentOverage));
         using var httpClient = new HttpClient();
         var requestUri = _configuration.GetValue<string>("CurrentPowerToGridUrl");
+        _logger.LogDebug("Using {uri} to get current overage.", requestUri);
         var response = await httpClient.GetAsync(
                 requestUri)
             .ConfigureAwait(false);
-        response.EnsureSuccessStatusCode();
+
+        if (!response.IsSuccessStatusCode)
+        {
+            _logger.LogError("Could not get current overage. {statusCode}, {reasonPhrase}", response.StatusCode, response.ReasonPhrase);
+            response.EnsureSuccessStatusCode();
+        }
+
         var result = await response.Content.ReadAsStringAsync().ConfigureAwait(false);
 
         var jsonPattern = _configuration.GetValue<string>("CurrentPowerToGridJsonPattern");