From 01923bb039624d1bb856ddbabdbfd7a96d7f5599 Mon Sep 17 00:00:00 2001 From: Rainnny7 Date: Thu, 19 Dec 2024 02:14:52 -0500 Subject: [PATCH] Fix a bug where requests would rely on the metrics server being online, and error if it wasn't which would prevent the request from being processed --- .../java/me/braydon/tether/service/DiscordService.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/API/src/main/java/me/braydon/tether/service/DiscordService.java b/API/src/main/java/me/braydon/tether/service/DiscordService.java index f8888ad..45115f5 100644 --- a/API/src/main/java/me/braydon/tether/service/DiscordService.java +++ b/API/src/main/java/me/braydon/tether/service/DiscordService.java @@ -2,6 +2,7 @@ package me.braydon.tether.service; import com.github.benmanes.caffeine.cache.Cache; import com.github.benmanes.caffeine.cache.Caffeine; +import io.questdb.cutlass.line.LineSenderException; import jakarta.annotation.PostConstruct; import kong.unirest.core.HttpResponse; import kong.unirest.core.HttpStatus; @@ -130,7 +131,12 @@ public final class DiscordService extends ListenerAdapter { long before = System.currentTimeMillis(); cachedUser = new CachedDiscordUser(getUser(snowflake, member != null), System.currentTimeMillis()); if (metricsEnabled) { - UserLookupTimingsMetric.track(System.currentTimeMillis() - before); + try { + UserLookupTimingsMetric.track(System.currentTimeMillis() - before); + } catch (LineSenderException | IllegalStateException ignored) { + // This can happen due to no metrics server being + // available, we can safely ignore it and continue + } } cachedUsers.put(snowflake, cachedUser); }