Merge pull request #1795 from pkuehnel/fix/GetCarsFromAccount

fix(TeslaFleetApiService): did return error on success status code
pkuehnel · Jan 30, 2025 · 883a19f · 883a19f
2 parents f94d9d9 + ab9765a
commit 883a19f
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/TeslaSolarCharger/Server/Services/TeslaFleetApiService.cs b/TeslaSolarCharger/Server/Services/TeslaFleetApiService.cs
@@ -1254,7 +1254,7 @@ public async Task<Fin<List<DtoTesla>>> GetAllCarsFromAccount()
                 return Fin<List<DtoTesla>>.Fail($"Could not deserialize response body");
             }
 
-            if (teslaBackendResult.StatusCode is >= HttpStatusCode.OK and < HttpStatusCode.MultipleChoices)
+            if (!(teslaBackendResult.StatusCode is >= HttpStatusCode.OK and < HttpStatusCode.MultipleChoices))
             {
                 logger.LogError("Error while getting all cars from account due to communication issue between Solar4Car Backend and Tesla: Underlaying Status code: {statusCode}; Underlaying Result: {jsonResult}", teslaBackendResult.StatusCode, teslaBackendResult.JsonResponse);
                 var excpetion = new HttpRequestException($"Requesting {requestUri} returned following statusCode: {teslaBackendResult.StatusCode} Underlaying result: {teslaBackendResult.JsonResponse}", null,