diff options
author | Lulonaut <lulonaut@tutanota.de> | 2023-04-22 13:34:50 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-04-22 21:34:50 +1000 |
commit | 02b53848959aa4f23e029d389c9fc93b2027eff1 (patch) | |
tree | e61f44e085426602620d76d17c16e4f278f7c572 | |
parent | 100ec81fc2f2af8f19cf034e2130ca7ba2661d4e (diff) | |
download | NotEnoughUpdates-02b53848959aa4f23e029d389c9fc93b2027eff1.tar.gz NotEnoughUpdates-02b53848959aa4f23e029d389c9fc93b2027eff1.tar.bz2 NotEnoughUpdates-02b53848959aa4f23e029d389c9fc93b2027eff1.zip |
Show invalid API key notification on Request failure (#674)
-rw-r--r-- | src/main/java/io/github/moulberry/notenoughupdates/util/ApiUtil.java | 33 |
1 files changed, 29 insertions, 4 deletions
diff --git a/src/main/java/io/github/moulberry/notenoughupdates/util/ApiUtil.java b/src/main/java/io/github/moulberry/notenoughupdates/util/ApiUtil.java index 28298fe0..3cb17772 100644 --- a/src/main/java/io/github/moulberry/notenoughupdates/util/ApiUtil.java +++ b/src/main/java/io/github/moulberry/notenoughupdates/util/ApiUtil.java @@ -1,5 +1,5 @@ /* - * Copyright (C) 2022 NotEnoughUpdates contributors + * Copyright (C) 2022-2023 NotEnoughUpdates contributors * * This file is part of NotEnoughUpdates. * @@ -24,6 +24,7 @@ import com.google.gson.JsonObject; import io.github.moulberry.notenoughupdates.NotEnoughUpdates; import io.github.moulberry.notenoughupdates.events.ProfileDataLoadedEvent; import net.minecraft.client.Minecraft; +import net.minecraft.util.EnumChatFormatting; import org.apache.commons.io.IOUtils; import org.apache.http.NameValuePair; import org.apache.http.client.utils.URIBuilder; @@ -50,13 +51,13 @@ import java.security.UnrecoverableKeyException; import java.security.cert.CertificateException; import java.time.Duration; import java.util.ArrayList; +import java.util.Arrays; import java.util.Collection; import java.util.Comparator; import java.util.HashMap; import java.util.List; import java.util.Map; import java.util.concurrent.CompletableFuture; -import java.util.concurrent.CompletionException; import java.util.concurrent.ExecutorService; import java.util.concurrent.Executors; import java.util.zip.GZIPInputStream; @@ -77,6 +78,7 @@ public class ApiUtil { } private static SSLContext ctx; private final Map<String, CompletableFuture<Void>> updateTasks = new HashMap<>(); + private static boolean notifiedOfInvalidApiKey; static { try { @@ -190,9 +192,9 @@ public class ApiUtil { private CompletableFuture<String> requestString0() { return buildUrl().thenApplyAsync(url -> { + InputStream inputStream = null; + URLConnection conn = null; try { - InputStream inputStream = null; - URLConnection conn = null; try { conn = url.openConnection(); if (conn instanceof HttpsURLConnection && ctx != null) { @@ -239,6 +241,29 @@ public class ApiUtil { } } } catch (IOException e) { + if (conn instanceof HttpURLConnection) { + try { + int code = ((HttpURLConnection) conn).getResponseCode(); + if (code == 403 && baseUrl != null && baseUrl.startsWith("https://api.hypixel.net/")) { + if (!notifiedOfInvalidApiKey) { + NotificationHandler.displayNotification(Arrays.asList( + EnumChatFormatting.RED + EnumChatFormatting.BOLD.toString() + "API request failed", + EnumChatFormatting.RED + "A Request failed because your API key is invalid/not present", + EnumChatFormatting.RED + "Please run " + EnumChatFormatting.BOLD + EnumChatFormatting.GOLD + + "/api new " + + EnumChatFormatting.RESET + EnumChatFormatting.RED + "to fix this.", + EnumChatFormatting.RED + + "If you don't do this, several API related features, such as the profile viewer, will not work.", + EnumChatFormatting.GRAY+"Press X on your keyboard to close this notification." + ), true, false); + notifiedOfInvalidApiKey = true; + } + return ""; + } + } catch (IOException ex) { + throw new RuntimeException(ex); + } + } throw new RuntimeException(e); // We can rethrow, since supplyAsync catches exceptions. } }, executorService).handle((obj, t) -> { |