Fix airport info query in Discord RP

nguyenquyhy · Dec 14, 2024 · dd041a6 · dd041a6
1 parent 087754f
commit dd041a6
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/FlightEvents.Client.Logics/DiscordRichPresentLogic.cs b/FlightEvents.Client.Logics/DiscordRichPresentLogic.cs
@@ -139,14 +139,15 @@ await geocodeExecutor.ExecuteAsync(async () =>
                             {
                                 try
                                 {
-                                    var dataString = await httpClient.GetStringAsync($"https://www.airport-data.com/api/ap_info.json?icao={icao}");
+                                    var dataString = await httpClient.GetStringAsync($"https://airport-data.com/api/ap_info.json?icao={icao}");
                                     var result = JsonConvert.DeserializeObject<AirportDataResult>(dataString);
                                     cachedAirports.TryAdd(icao, result);
                                     country = result.country;
                                 }
                                 catch (Exception ex)
                                 {
-                                    logger.LogError(ex, $"Cannot get airport information of {icao}!");
+                                    logger.LogError(ex, "Cannot get airport information of {icao}!", icao);
+                                    // TODO: improve error handling so there is some exponential backoff for this query
                                 }
                             }
                         }