fix: make lookup more reliable

common/src/main/java/net/blockhost/anarchymod/Domains.java

@@ -8,11 +8,15 @@ import java.net.URI;
 import java.net.http.HttpClient;
 import java.net.http.HttpRequest;
 import java.net.http.HttpResponse;
+import java.time.Duration;
 import java.util.HashSet;
+import java.util.Locale;
 import java.util.Set;
+import java.util.logging.Logger;
 
 public class Domains {
 
+    private static final Logger LOGGER = Logger.getLogger("AnarchyMod-Domains");
     private static final String DOMAINS_URL = "https://www.6b6t.org/api/anarchy-mod.json";
 
     private static final Set<String> DEFAULT = Set.of(
@@ -25,37 +29,36 @@ public class Domains {
     private static final Gson gson = new Gson();
 
     static {
-        DEFAULT.forEach(d -> domains.add(d.toLowerCase()));
+        DEFAULT.forEach(d -> domains.add(d.toLowerCase(Locale.ROOT)));
         loadRemote();
     }
 
     private static void loadRemote() {
-        try {
-            HttpClient client = HttpClient.newHttpClient();
+        try (HttpClient client = HttpClient.newHttpClient()) {
             HttpRequest request = HttpRequest.newBuilder()
                 .uri(URI.create(DOMAINS_URL))
+                .timeout(Duration.ofSeconds(15))
                 .GET()
                 .build();
 
-            HttpResponse<String> response =
-                    client.send(request, HttpResponse.BodyHandlers.ofString());
+            HttpResponse<String> response = client.send(request, HttpResponse.BodyHandlers.ofString());
 
             JsonObject json = gson.fromJson(response.body(), JsonObject.class);
             JsonArray array = json.getAsJsonArray("domains");
             if (array == null) return;
 
             array.forEach(element ->
-                    domains.add(element.getAsString().toLowerCase())
+                domains.add(element.getAsString().toLowerCase(Locale.ROOT))
             );
-
-        } catch (Exception ignored) {
+        } catch (Exception e) {
+            LOGGER.warning("Failed to load domains from remote, using defaults: " + e.getMessage());
         }
     }
 
     public static boolean contains(String input) {
         if (input == null) return false;
 
-        String domain = input.toLowerCase();
+        String domain = input.toLowerCase(Locale.ROOT);
 
         if (domain.startsWith("*.")) {
             domain = domain.substring(2);
@@ -67,7 +70,6 @@ public class Domains {
         }
 
         for (String entry : domains) {
-
             if (entry.startsWith("*.")) {
                 String base = entry.substring(2);
                 if (domain.equals(base) || domain.endsWith("." + base)) {