2.0.9

Bumps com.velocitypowered:velocity-api from 3.4.0-SNAPSHOT to 3.5.0-SNAPSHOT.

---
updated-dependencies:
- dependency-name: com.velocitypowered:velocity-api
  dependency-version: 3.5.0-SNAPSHOT
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

.github/dependabot.yml

@@ -0,0 +1,19 @@
+version: 2
+updates:
+  - package-ecosystem: "gradle"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    open-pull-requests-limit: 10
+    ignore:
+      - dependency-name: "*"
+        update-types: ["version-update:semver-major"]
+
+  - package-ecosystem: "github-actions"
+    directory: "/"
+    schedule:
+      interval: "weekly"
+    open-pull-requests-limit: 10
+    ignore:
+      - dependency-name: "*"
+        update-types: ["version-update:semver-major"]

.github/workflows/dependabot.yml

@@ -0,0 +1,22 @@
+on:
+  pull_request:
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  dependabot:
+    runs-on: ubuntu-latest
+    if: github.event.pull_request.user.login == 'dependabot[bot]'
+    steps:
+      - id: metadata
+        uses: dependabot/fetch-metadata@21025c705c08248db411dc16f3619e6b5f9ea21a
+        with:
+          github-token: "${{ secrets.GITHUB_TOKEN }}"
+
+      - if: steps.metadata.outputs.update-type != 'version-update:semver-major'
+        run: gh pr merge --auto --squash "$PR_URL"
+        env:
+          PR_URL: ${{ github.event.pull_request.html_url }}
+          GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/gradle.yml

@@ -1,5 +1,3 @@
-name: Build Plugin
-
 on:
   workflow_dispatch:
   push:

README.md

@@ -24,6 +24,11 @@
 - [x] Modular multi-platform support
 - [x] JSON5 based configuration
 
+<h2>Requirements</h2>
+
+- Eagler(craft)XServer v1.0.2 or newer
+- JDK 21 or newer
+
 <h2>Download</h2>
 The latest release can be found at <b><a href="https://github.com/WebMCDevelopment/originblacklist/releases/latest/">https://github.com/WebMCDevelopment/originblacklist/releases/latest/</a></b>
 

build.gradle.kts

@@ -14,7 +14,7 @@ val PLUGIN_NAME = "OriginBlacklist"
 val PLUGIN_IDEN = "originblacklist"
 val PLUGIN_DOMN = "xyz.webmc.$PLUGIN_IDEN"
 val PLUGIN_DESC = "An eaglercraft client blacklist plugin."
-val PLUGIN_VERS = "2.0.7"
+val PLUGIN_VERS = "2.0.9"
 val PLUGIN_SITE = "https://github.com/WebMCDevelopment/$PLUGIN_IDEN"
 val PLUGIN_DEPA = listOf("EaglercraftXServer")
 val PLUGIN_DEPB = listOf("EaglercraftXServer")
@@ -42,7 +42,7 @@ val EAGXS_VER = "1.0.8"
 
 plugins {
   id("java")
-  id("com.gradleup.shadow") version "9.3.1"
+  id("com.gradleup.shadow") version "9.3.0"
   id("com.palantir.git-version") version "4.2.0"
   id("xyz.jpenilla.run-paper") version "3.0.2"
   id("xyz.jpenilla.run-waterfall") version "3.0.2"
@@ -65,7 +65,7 @@ repositories {
 }
 
 dependencies {
-  compileOnly("com.velocitypowered:velocity-api:3.4.0-SNAPSHOT")
+  compileOnly("com.velocitypowered:velocity-api:3.5.0-SNAPSHOT")
   compileOnly("org.bukkit:bukkit:1.8-R0.1-SNAPSHOT")
   compileOnly("net.md-5:bungeecord-api:1.21-R0.5-SNAPSHOT")
   compileOnly("net.lax1dude.eaglercraft.backend:api-velocity:1.0.0")
@@ -91,7 +91,7 @@ sourceSets {
 }
 
 java {
-  toolchain.languageVersion.set(JavaLanguageVersion.of(17))
+  toolchain.languageVersion.set(JavaLanguageVersion.of(21))
 }
 
 val BUILD_PROPS = mapOf(
@@ -113,7 +113,7 @@ val BUILD_PROPS = mapOf(
 
 tasks.withType<JavaCompile>().configureEach {
   options.encoding = "UTF-8"
-  options.release.set(17)
+  options.release.set(21)
 }
 
 tasks.withType<ProcessResources>().configureEach {

src/main/java/xyz/webmc/originblacklist/base/OriginBlacklist.java

@@ -7,6 +7,7 @@ import xyz.webmc.originblacklist.base.enums.EnumLogLevel;
 import xyz.webmc.originblacklist.base.events.OriginBlacklistLoginEvent;
 import xyz.webmc.originblacklist.base.events.OriginBlacklistMOTDEvent;
 import xyz.webmc.originblacklist.base.http.OriginBlacklistRequestHandler;
+import xyz.webmc.originblacklist.base.metrics.GenericMetricsAdapter;
 import xyz.webmc.originblacklist.base.util.BuildInfo;
 import xyz.webmc.originblacklist.base.util.IOriginBlacklistPlugin;
 import xyz.webmc.originblacklist.base.util.OPlayer;
@@ -15,6 +16,7 @@ import xyz.webmc.originblacklist.base.util.UpdateChecker;
 import java.io.InputStream;
 import java.io.OutputStream;
 import java.net.HttpURLConnection;
+import java.net.URI;
 import java.net.URL;
 import java.net.URLDecoder;
 import java.nio.charset.StandardCharsets;
@@ -26,7 +28,9 @@ import java.nio.file.StandardOpenOption;
 import java.time.Instant;
 import java.util.ArrayList;
 import java.util.Base64;
+import java.util.HashMap;
 import java.util.List;
+import java.util.Map;
 import java.util.concurrent.TimeUnit;
 
 import de.marhali.json5.Json5;
@@ -38,8 +42,11 @@ import inet.ipaddr.IPAddressString;
 import net.kyori.adventure.text.Component;
 import net.kyori.adventure.text.minimessage.MiniMessage;
 import net.kyori.adventure.text.serializer.legacy.LegacyComponentSerializer;
+import net.lax1dude.eaglercraft.backend.server.api.IBasePlayer;
+import net.lax1dude.eaglercraft.backend.server.api.IEaglerPlayer;
 import net.lax1dude.eaglercraft.backend.server.api.IEaglerXServerAPI;
 import net.lax1dude.eaglercraft.backend.server.api.query.IMOTDConnection;
+import org.bstats.charts.AdvancedPie;
 import org.semver4j.Semver;
 
 @SuppressWarnings({ "rawtypes" })
@@ -55,6 +62,7 @@ public final class OriginBlacklist {
 
   private final IOriginBlacklistPlugin plugin;
   private final OriginBlacklistConfig config;
+  private final GenericMetricsAdapter metrics;
   private final Json5 json5;
   private String updateURL;
   private Path jarFile;
@@ -62,6 +70,7 @@ public final class OriginBlacklist {
   public OriginBlacklist(final IOriginBlacklistPlugin plugin) {
     this.plugin = plugin;
     this.config = new OriginBlacklistConfig(this);
+    this.metrics = plugin.getMetrics();
     this.json5 = Json5.builder(builder -> builder.prettyPrinting().indentFactor(0).build());
   }
 
@@ -73,8 +82,24 @@ public final class OriginBlacklist {
     if (this.isBlacklistAPIEnabled()) {
       OriginBlacklistRequestHandler.register(this);
     }
+    this.metrics.addCustomChart(new AdvancedPie("player_types", () -> {
+      final Map<String, Integer> playerMap = new HashMap<>();
+      for (final Object player : this.getEaglerAPI().getAllPlayers()) {
+        if (player instanceof IBasePlayer bPlayer) {
+          final String key = (bPlayer instanceof IEaglerPlayer) ? "Eagler" : "Java";
+          playerMap.put(key, playerMap.getOrDefault(key, 0) + 1);
+        }
+      }
+      return playerMap;
+    }));
     this.plugin.log(EnumLogLevel.INFO, "Initialized Plugin");
     this.plugin.log(EnumLogLevel.DEBUG, "Commit " + COMMIT_L);
+    if (this.isMetricsEnabled()) {
+      this.metrics.start();
+    }
+    this.plugin.scheduleRepeat(() -> {
+      this.plugin.log(EnumLogLevel.INFO, String.valueOf(this.isMetricsEnabled()));
+    }, 1, TimeUnit.SECONDS);
   }
 
   public final void shutdown() {
@@ -82,6 +107,9 @@ public final class OriginBlacklist {
     if (this.isBlacklistAPIEnabled()) {
       OriginBlacklistRequestHandler.unRegister(this);
     }
+    if (this.isMetricsEnabled()) {
+      this.metrics.shutdown();
+    }
     this.plugin.shutdown();
   }
 
@@ -94,6 +122,14 @@ public final class OriginBlacklist {
       }
     } catch (final Throwable t) {
     }
+    try {
+      if (this.isMetricsEnabled()) {
+        this.metrics.start();
+      } else {
+        this.metrics.shutdown();
+      }
+    } catch (final Throwable t) {
+    }
   }
 
   public final void handleLogin(final OriginBlacklistLoginEvent event) {
@@ -114,10 +150,10 @@ public final class OriginBlacklist {
       }
       this.plugin.kickPlayer(this.getBlacklistedComponent("kick", blacklisted.getArrayString(),
           blacklisted.getAltString(), blacklisted.getString(), "not allowed", "not allowed on the server",
-          blacklisted_value, blacklisted.getActionString()), event);
+          blacklisted_value, blacklisted.getActionString(), false), event);
       this.sendWebhooks(event, blacklisted);
       final String name = player.getName();
-      if (isNonNull(name)) {
+      if (isNonNullStr(name)) {
         this.plugin.log(EnumLogLevel.INFO, "Prevented blacklisted player " + name + " from joining.");
         this.updateLogFile(event, blacklisted);
       }
@@ -136,9 +172,12 @@ public final class OriginBlacklist {
       } else {
         blacklisted_value = UNKNOWN_STR;
       }
-      this.plugin.setMOTD(this.getBlacklistedComponent("motd", blacklisted.getArrayString(), blacklisted.getAltString(),
+      if (this.isMOTDEnabled()) {
-          blacklisted.getString(), "blacklisted", "blacklisted from the server", blacklisted_value,
+        this.plugin
-          blacklisted.getActionString()), event);
+            .setMOTD(this.getBlacklistedComponent("motd", blacklisted.getArrayString(), blacklisted.getAltString(),
+                blacklisted.getString(), "blacklisted", "blacklisted from the server", blacklisted_value,
+                blacklisted.getActionString(), true), event);
+      }
     }
   }
 
@@ -150,6 +189,14 @@ public final class OriginBlacklist {
     return this.config.getBoolean("bStats");
   }
 
+  public final boolean isMOTDEnabled() {
+    return this.config.getBoolean("motd.enabled");
+  }
+
+  public final boolean isWebhooksEnabled() {
+    return this.config.getBoolean("discord.webhook.enabled");
+  }
+
   public final boolean isLogFileEnabled() {
     return this.config.getBoolean("logFile");
   }
@@ -165,13 +212,18 @@ public final class OriginBlacklist {
   public final void setEaglerMOTD(final Component comp, final OriginBlacklistMOTDEvent event) {
     final IMOTDConnection conn = event.getEaglerEvent().getMOTDConnection();
     final List<String> lst = new ArrayList<>();
-    for (String ln : getComponentString(comp).split("\n")) {
+    for (final String ln : getComponentString(comp).split("\n")) {
       lst.add(ln);
     }
+    final List<String> pLst = new ArrayList<>();
+    for (final Json5Element ln : this.config.getArray("motd.players.hover").getAsJson5Array()) {
+      pLst.add(getLegacyFromMiniMessage(
+          ln.getAsString().replaceAll("%discord_invite%", this.config.getString("discord.invite"))));
+    }
     conn.setServerMOTD(lst);
-    conn.setPlayerTotal(0);
+    conn.setPlayerTotal(this.config.getInteger("motd.players.online"));
-    conn.setPlayerUnlimited();
+    conn.setPlayerMax(this.config.getInteger("motd.players.max"));
-    conn.setPlayerList(List.of());
+    conn.setPlayerList(pLst);
     conn.setServerIcon(this.config.getIconBytes());
     conn.sendToUser();
     conn.disconnect();
@@ -182,7 +234,7 @@ public final class OriginBlacklist {
       this.plugin.runAsync(() -> {
         this.updateURL = UpdateChecker.checkForUpdates(PLUGIN_REPO, this.plugin.getPluginVersion(),
             this.config.getBoolean("update_checker.allow_snapshots"));
-        if (isNonNull((this.updateURL))) {
+        if (isNonNullStr((this.updateURL))) {
           action1.run();
           return;
         }
@@ -195,7 +247,7 @@ public final class OriginBlacklist {
 
   public final void updatePlugin(final Runnable action1, final Runnable action2) {
     try {
-      final URL url = new URL(this.updateURL);
+      final URL url = (new URI(this.updateURL)).toURL();
       final Path jar = this.jarFile;
       final Path bak = jar.resolveSibling(jar.getFileName().toString() + ".bak");
       final Path upd = jar
@@ -225,8 +277,8 @@ public final class OriginBlacklist {
         action1.run();
         return;
       } catch (final Throwable t) {
-        t.printStackTrace();
         Files.move(bak, jar, StandardCopyOption.REPLACE_EXISTING);
+        throw t;
       }
     } catch (final Throwable t) {
       t.printStackTrace();
@@ -249,7 +301,7 @@ public final class OriginBlacklist {
     final boolean whitelist = this.config.getBoolean("blacklist_to_whitelist");
     EnumBlacklistType type = EnumBlacklistType.NONE;
 
-    if (isNonNull(origin)) {
+    if (isNonNullStr(origin)) {
       if (whitelist && !type.isBlacklisted())
         type = EnumBlacklistType.ORIGIN;
       for (final Json5Element element : this.config.getArray("blacklist.origins").getAsJson5Array()) {
@@ -265,7 +317,7 @@ public final class OriginBlacklist {
       return whitelist ? EnumBlacklistType.NONE : EnumBlacklistType.ORIGIN;
     }
 
-    if (isNonNull(brand)) {
+    if (isNonNullStr(brand)) {
       if (whitelist && !type.isBlacklisted())
         type = EnumBlacklistType.BRAND;
       for (final Json5Element element : this.config.getArray("blacklist.brands")) {
@@ -279,7 +331,7 @@ public final class OriginBlacklist {
       }
     }
 
-    if (isNonNull(name)) {
+    if (isNonNullStr(name)) {
       if (whitelist && !type.isBlacklisted())
         type = EnumBlacklistType.NAME;
       for (final Json5Element element : this.config.getArray("blacklist.player_names")) {
@@ -293,7 +345,7 @@ public final class OriginBlacklist {
       }
     }
 
-    if (isNonNull(addr)) {
+    if (isNonNullStr(addr)) {
       if (whitelist && !type.isBlacklisted())
         type = EnumBlacklistType.ADDR;
       for (final Json5Element element : this.config.getArray("blacklist.ip_addresses")) {
@@ -345,8 +397,8 @@ public final class OriginBlacklist {
 
   private final Component getBlacklistedComponent(final String type, final String id, final String blockType,
       final String blockTypeAlt, final String notAllowed, final String notAllowedAlt, final String blockValue,
-      final String action) {
+      final String action, final boolean isMOTD) {
-    final Json5Array arr = this.config.getArray("messages." + type);
+    final Json5Array arr = this.config.getArray(isMOTD ? (type + ".text") : ("messages." + type));
     final StringBuilder sb = new StringBuilder();
     for (int i = 0; i < arr.size(); i++) {
       if (i > 0)
@@ -359,21 +411,26 @@ public final class OriginBlacklist {
         .replaceAll("%block_type%", blockType)
         .replaceAll("%not_allowed%", notAllowed)
         .replaceAll("%not_allowed_alt%", notAllowedAlt)
-        .replaceAll("%blocked_value%", blockValue);
+        .replaceAll("%blocked_value%", blockValue)
+        .replaceAll("%discord_invite%", this.config.getString("discord.invite"));
     return MiniMessage.miniMessage().deserialize(str);
   }
 
   private final void sendWebhooks(final OriginBlacklistLoginEvent event, final EnumBlacklistType type) {
-    if (this.config.getBoolean("discord.enabled")) {
+    if (this.isWebhooksEnabled()) {
       final OPlayer player = event.getPlayer();
       final EnumConnectionType connType = event.getConnectionType();
-      /* final String userAgent;
+      /*
-      if (connType == EnumConnectionType.EAGLER) {
+       * final String userAgent;
-        final IEaglerLoginConnection loginConn = event.getEaglerEvent().getLoginConnection();
+       * if (connType == EnumConnectionType.EAGLER) {
-        userAgent = loginConn.getWebSocketHeader(EnumWebSocketHeader.HEADER_USER_AGENT);
+       * final IEaglerLoginConnection loginConn =
-      } else {
+       * event.getEaglerEvent().getLoginConnection();
-        userAgent = UNKNOWN_STR;
+       * userAgent =
-      } */
+       * loginConn.getWebSocketHeader(EnumWebSocketHeader.HEADER_USER_AGENT);
+       * } else {
+       * userAgent = UNKNOWN_STR;
+       * }
+       */
       final byte[] payload = String.format(
           """
                 {
@@ -426,7 +483,7 @@ public final class OriginBlacklist {
       for (final Json5Element element : arr) {
         this.plugin.runAsync(() -> {
           try {
-            final URL url = new URL(element.getAsString());
+            final URL url = (new URI(element.getAsString())).toURL();
             final HttpURLConnection conn = (HttpURLConnection) url.openConnection();
             conn.setRequestMethod("POST");
             conn.setRequestProperty("Content-Type", "application/json");
@@ -466,17 +523,17 @@ public final class OriginBlacklist {
   private final void updateLogFile(final OriginBlacklistLoginEvent event, final EnumBlacklistType type) {
     if (this.isLogFileEnabled()) {
       final OPlayer player = event.getPlayer();
-      final String txt = Instant.now() + " - [player=" + player.getName() + "," + "blacklist_reason=" + type.toString() + "]";
+      final String txt = Instant.now() + " - [player=" + player.getName() + "," + "blacklist_reason=" + type.toString()
+          + "]";
       final Path dir = Paths.get(this.getDataDir());
       try {
         Files.createDirectories(dir);
         Files.writeString(
-          dir.resolve("blacklist.log"),
+            dir.resolve("blacklist.log"),
-          txt + "\n",
+            txt + "\n",
-          StandardOpenOption.CREATE,
+            StandardOpenOption.CREATE,
-          StandardOpenOption.WRITE,
+            StandardOpenOption.WRITE,
-          StandardOpenOption.APPEND
+            StandardOpenOption.APPEND);
-        );
       } catch (final Throwable t) {
         t.printStackTrace();
       }
@@ -499,7 +556,7 @@ public final class OriginBlacklist {
     return BuildInfo.get("plugin_name") + "/" + BuildInfo.get("plugin_vers") + "+" + BuildInfo.get("git_cm_hash");
   }
 
-  public static final boolean isNonNull(final String str) {
+  public static final boolean isNonNullStr(final String str) {
     return str != null && !str.isEmpty() && !str.isBlank() && !str.equals("null");
   }
 

src/main/java/xyz/webmc/originblacklist/base/command/OriginBlacklistCommand.java

@@ -54,7 +54,7 @@ public class OriginBlacklistCommand implements ICommand {
           } else {
             ctx.reply(NO_PERMISSION);
           }
-        } else if ((add || remove) && OriginBlacklist.isNonNull(argB)) {
+        } else if ((add || remove) && OriginBlacklist.isNonNullStr(argB)) {
           if ((add && ctx.hasPermission("originblacklist.command.add"))
               || (remove && ctx.hasPermission("originblacklist.command.add"))) {
             final String arrName;
@@ -69,7 +69,7 @@ public class OriginBlacklistCommand implements ICommand {
             } else {
               arrName = null;
             }
-            if (OriginBlacklist.isNonNull(arrName)) {
+            if (OriginBlacklist.isNonNullStr(arrName)) {
               final String arrPath = "blacklist." + arrName;
               final Json5Array arr = config.getArray(arrPath);
               if (add) {
@@ -95,7 +95,7 @@ public class OriginBlacklistCommand implements ICommand {
           } else {
             ctx.reply(NO_PERMISSION);
           }
-        } else if ("test".equalsIgnoreCase(command) && OriginBlacklist.isNonNull(argA)) {
+        } else if ("test".equalsIgnoreCase(command) && OriginBlacklist.isNonNullStr(argA)) {
           if (ctx.hasPermission("originblacklist.command.test")) {
             if (this.isBlacklisted(argA)) {
               ctx.reply("<green>" + argA + " is on the blacklist.</green>");

src/main/java/xyz/webmc/originblacklist/base/config/OriginBlacklistConfig.java

@@ -22,7 +22,8 @@ import de.marhali.json5.Json5Object;
 import de.marhali.json5.Json5Primitive;
 
 public final class OriginBlacklistConfig {
-  private static final Json5Object DEFAULT_CONFIG = getDefaultConfig();
+  public static final Json5Object DEFAULT_CONFIG = getDefaultConfig();
+  public static final int LATEST_CONFIG_VERSION = 2;
 
   private final Json5 json5;
   private final File file;
@@ -41,7 +42,7 @@ public final class OriginBlacklistConfig {
         .writeComments()
         .prettyPrinting()
         .build());
-    
+
     this.file = new File(plugin.getDataDir() + "/config.json5");
     this.filePath = file.toPath();
     this.iconFile = new File(plugin.getDataDir() + "/blacklisted.png");
@@ -73,6 +74,7 @@ public final class OriginBlacklistConfig {
       Json5Element parsed = this.json5.parse(text);
       if (parsed instanceof Json5Object) {
         this.config = (Json5Object) parsed;
+        this.config = OriginBlacklistConfigTransformer.transformConfig(this.config);
         merge(this.config, DEFAULT_CONFIG);
       } else {
         throw new IOException("Config must be an object!");
@@ -130,7 +132,7 @@ public final class OriginBlacklistConfig {
   public final Json5Element get(final String key) {
     Json5Element element = null;
 
-    if (this.config != null && OriginBlacklist.isNonNull(key)) {
+    if (this.config != null && OriginBlacklist.isNonNullStr(key)) {
       element = this.config;
       final String[] parts = splitPath(key);
 
@@ -262,18 +264,29 @@ public final class OriginBlacklistConfig {
     kick.add("%action%");
     kick.add("");
     kick.add("<aqua>Think this is a mistake? Join our discord:</aqua>");
-    kick.add("<blue>discord.gg/changethisintheconfig</blue>");
+    kick.add("<blue>%discord_invite%</blue>");
     addJSONObj(mObj, "kick", kick, null);
-    final Json5Array motd = new Json5Array();
-    motd.add("<red>This %block_type% is %not_allowed%!</red>");
-    motd.add("<dark_gray>»</dark_gray> <gray>%blocked_value%</gray>");
-    addJSONObj(mObj, "motd", motd, null);
     final Json5Object actions = new Json5Object();
     actions.add("generic", Json5Primitive.fromString("<gold>Please switch to a different %block_type%.</gold>"));
     actions.add("player_name", Json5Primitive.fromString("<gold>Please change your %block_type%.</gold>"));
     actions.add("ip_address", Json5Primitive.fromString("<gold>Please contact staff for assistance.</gold>"));
     addJSONObj(mObj, "actions", actions, null);
     addJSONObj(obj, "messages", mObj, null);
+    final Json5Object nObj = new Json5Object();
+    addJSONObj(nObj, "enabled", Json5Primitive.fromBoolean(true), null);
+    final Json5Array mArr = new Json5Array();
+    mArr.add("<red>This %block_type% is %not_allowed%!</red>");
+    mArr.add("<dark_gray>»</dark_gray> <gray>%blocked_value%</gray>");
+    addJSONObj(nObj, "text", mArr, null);
+    final Json5Object mPlayers = new Json5Object();
+    addJSONObj(mPlayers, "online", Json5Primitive.fromNumber(0), null);
+    addJSONObj(mPlayers, "max", Json5Primitive.fromNumber(0), null);
+    final Json5Array hArr = new Json5Array();
+    hArr.add("<blue>Join our discord</blue>");
+    hArr.add("<blue>%discord_invite%</blue>");
+    addJSONObj(mPlayers, "hover", hArr, null);
+    addJSONObj(nObj, "players", mPlayers, null);
+    addJSONObj(obj, "motd", nObj, null);
     final Json5Object bObj = new Json5Object();
     final Json5Array origins = new Json5Array();
     origins.add(".*eaglerhackedclients\\.vercel\\.app.*");
@@ -302,9 +315,12 @@ public final class OriginBlacklistConfig {
     addJSONObj(bObj, "ip_addresses", ips, null);
     addJSONObj(obj, "blacklist", bObj, null);
     final Json5Object dObj = new Json5Object();
-    addJSONObj(dObj, "enabled", Json5Primitive.fromBoolean(false), null);
+    addJSONObj(dObj, "invite", Json5Primitive.fromString("discord.gg/changeme"), null);
-    addJSONObj(dObj, "webhook_urls", new Json5Array(), null);
+    final Json5Object webhook = new Json5Object();
-    addJSONObj(dObj, "send_ips", Json5Primitive.fromBoolean(true), null);
+    addJSONObj(webhook, "enabled", Json5Primitive.fromBoolean(false), null);
+    addJSONObj(webhook, "webhook_urls", new Json5Array(), null);
+    addJSONObj(webhook, "send_ips", Json5Primitive.fromBoolean(true), null);
+    addJSONObj(dObj, "webhook", webhook, null);
     addJSONObj(obj, "discord", dObj, null);
     final Json5Object uObj = new Json5Object();
     addJSONObj(uObj, "enabled", Json5Primitive.fromBoolean(true), null);
@@ -317,7 +333,7 @@ public final class OriginBlacklistConfig {
     addJSONObj(obj, "block_undefined_origin", Json5Primitive.fromBoolean(false), null);
     addJSONObj(obj, "bStats", Json5Primitive.fromBoolean(true), null);
     addJSONObj(obj, "logFile", Json5Primitive.fromBoolean(true), null);
-    addJSONObj(obj, "config_version", Json5Primitive.fromNumber(1), "DO NOT CHANGE");
+    addJSONObj(obj, "config_version", Json5Primitive.fromNumber(LATEST_CONFIG_VERSION), "DO NOT CHANGE");
     return obj;
   }
 
@@ -477,7 +493,7 @@ public final class OriginBlacklistConfig {
 
   private static final void addJSONObj(final Json5Object obj, final String key, final Json5Element value,
       final String comment) {
-    if (OriginBlacklist.isNonNull(comment)) {
+    if (OriginBlacklist.isNonNullStr(comment)) {
       value.setComment(comment);
     }
     obj.add(key, value);
@@ -486,7 +502,7 @@ public final class OriginBlacklistConfig {
   private static final String[] splitPath(final String key) {
     final String[] ret;
 
-    if (OriginBlacklist.isNonNull(key)) {
+    if (OriginBlacklist.isNonNullStr(key)) {
       ret = key.split("\\.");
     } else {
       ret = new String[0];

src/main/java/xyz/webmc/originblacklist/base/config/OriginBlacklistConfigTransformer.java

@@ -0,0 +1,39 @@
+package xyz.webmc.originblacklist.base.config;
+
+import de.marhali.json5.Json5Array;
+import de.marhali.json5.Json5Object;
+import de.marhali.json5.Json5Primitive;
+
+public final class OriginBlacklistConfigTransformer {
+  public static final Json5Object transformConfig(final Json5Object config) {
+    final Json5Object obj = config.deepCopy();
+    final int ver = obj.get("config_version").getAsInt();
+    if (ver <= 1) {
+      final Json5Object mObj = obj.get("messages").getAsJson5Object().deepCopy();
+      final Json5Array motd = mObj.get("motd").getAsJson5Array().deepCopy();
+      mObj.remove("motd");
+      final Json5Object nObj = new Json5Object();
+      nObj.add("text", motd);
+      obj.remove("messages");
+      obj.add("messages", mObj);
+      obj.add("motd", nObj);
+      final Json5Object dObj = obj.get("discord").getAsJson5Object().deepCopy();
+      final Json5Primitive wEnabled = dObj.get("enabled").getAsJson5Primitive();
+      final Json5Array wURLS = dObj.get("webhook_urls").getAsJson5Array().deepCopy();
+      final Json5Primitive wAddrs = dObj.get("send_ips").getAsJson5Primitive();
+      dObj.remove("enabled");
+      dObj.remove("webhook_urls");
+      dObj.remove("send_ips");
+      final Json5Object oObj = new Json5Object();
+      oObj.add("enabled", wEnabled);
+      oObj.add("webhook_urls", wURLS);
+      oObj.add("send_ips", wAddrs);
+      dObj.add("webhook", oObj);
+      obj.remove("discord");
+      obj.add("discord", dObj);
+    }
+    obj.remove("config_version");
+    obj.add("config_version", Json5Primitive.fromNumber(OriginBlacklistConfig.LATEST_CONFIG_VERSION));
+    return obj;
+  }
+}

src/main/java/xyz/webmc/originblacklist/base/enums/EnumBlacklistType.java

@@ -18,7 +18,7 @@ public enum EnumBlacklistType {
     this.str = str;
     this.alt = alt;
     this.arr = arr;
-    this.act = OriginBlacklist.isNonNull(act) ? act : OriginBlacklist.GENERIC_STR.toLowerCase();
+    this.act = OriginBlacklist.isNonNullStr(act) ? act : OriginBlacklist.GENERIC_STR.toLowerCase();
   }
 
   public final String getString() {

src/main/java/xyz/webmc/originblacklist/base/http/OriginBlacklistRequestHandler.java

@@ -8,12 +8,12 @@ import net.lax1dude.eaglercraft.backend.server.api.webserver.IRequestContext;
 import net.lax1dude.eaglercraft.backend.server.api.webserver.IRequestHandler;
 import net.lax1dude.eaglercraft.backend.server.api.webserver.RouteDesc;
 
-public class OriginBlacklistRequestHandler implements IRequestHandler {
+public final class OriginBlacklistRequestHandler implements IRequestHandler {
   private static final RouteDesc route = RouteDesc.create("/originblacklist/v2/");
   private final OriginBlacklist plugin;
 
   public OriginBlacklistRequestHandler(final OriginBlacklist plugin) {
-    this.plugin = plugin; 
+    this.plugin = plugin;
   }
 
   @Override

src/main/java/xyz/webmc/originblacklist/base/metrics/GenericMetricsAdapter.java

@@ -0,0 +1,21 @@
+package xyz.webmc.originblacklist.base.metrics;
+
+import java.util.ArrayList;
+import java.util.List;
+
+import org.bstats.charts.CustomChart;
+
+public abstract class GenericMetricsAdapter {
+  protected final List<CustomChart> charts;
+
+  protected GenericMetricsAdapter() {
+    this.charts = new ArrayList<>();
+  }
+  
+  public final void addCustomChart(final CustomChart chart) {
+    this.charts.add(chart);
+  }
+
+  public abstract void start();
+  public abstract void shutdown();
+}

src/main/java/xyz/webmc/originblacklist/base/util/IOriginBlacklistPlugin.java

@@ -3,6 +3,7 @@ package xyz.webmc.originblacklist.base.util;
 import xyz.webmc.originblacklist.base.enums.EnumLogLevel;
 import xyz.webmc.originblacklist.base.events.OriginBlacklistLoginEvent;
 import xyz.webmc.originblacklist.base.events.OriginBlacklistMOTDEvent;
+import xyz.webmc.originblacklist.base.metrics.GenericMetricsAdapter;
 
 import java.nio.file.Path;
 import java.util.concurrent.TimeUnit;
@@ -19,6 +20,8 @@ public interface IOriginBlacklistPlugin {
 
   public Path getPluginJarPath();
 
+  public GenericMetricsAdapter getMetrics();
+
   public IEaglerXServerAPI getEaglerAPI();
 
   public void log(final EnumLogLevel level, final String txt);

src/main/java/xyz/webmc/originblacklist/base/util/OPlayer.java

@@ -85,7 +85,7 @@ public final class OPlayer {
   }
 
   private static final String formatIPAddress(String addr) {
-    if (OriginBlacklist.isNonNull(addr)) {
+    if (OriginBlacklist.isNonNullStr(addr)) {
       if (addr.startsWith("/")) {
         addr = addr.substring(1);
       }

src/main/java/xyz/webmc/originblacklist/base/util/UpdateChecker.java

@@ -5,6 +5,7 @@ import xyz.webmc.originblacklist.base.OriginBlacklist;
 import java.io.BufferedReader;
 import java.io.InputStreamReader;
 import java.net.HttpURLConnection;
+import java.net.URI;
 import java.net.URL;
 
 import de.marhali.json5.Json5;
@@ -20,7 +21,7 @@ public final class UpdateChecker {
   public static final String checkForUpdates(final String repo, final Semver currentVersion,
       final boolean allowSnapshots) {
     try {
-      final URL url = new URL("https://api.github.com/repos/" + repo + "/releases");
+      final URL url = (new URI("https://api.github.com/repos/" + repo + "/releases")).toURL();
       final HttpURLConnection conn = (HttpURLConnection) url.openConnection();
       conn.setRequestMethod("GET");
       conn.setConnectTimeout(5000);
@@ -75,7 +76,7 @@ public final class UpdateChecker {
             comm = "";
           }
           if (ver.isGreaterThan(currentVersion) || (allowSnapshots && currentVersion.diff(ver) == VersionDiff.BUILD
-              && OriginBlacklist.isNonNull(comm) && !BuildInfo.get("git_cm_hash").startsWith(comm))) {
+              && OriginBlacklist.isNonNullStr(comm) && !BuildInfo.get("git_cm_hash").startsWith(comm))) {
             element = obj.get("assets");
             if (element instanceof Json5Array) {
               final Json5Array aArr = element.getAsJson5Array();

src/main/java/xyz/webmc/originblacklist/bukkit/BukkitMetricsAdapter.java

@@ -0,0 +1,35 @@
+package xyz.webmc.originblacklist.bukkit;
+
+import xyz.webmc.originblacklist.base.OriginBlacklist;
+import xyz.webmc.originblacklist.base.metrics.GenericMetricsAdapter;
+
+import org.bstats.charts.CustomChart;
+import org.bstats.bukkit.Metrics;
+
+public final class BukkitMetricsAdapter extends GenericMetricsAdapter {
+  private final OriginBlacklistBukkit plugin;
+  private Metrics metrics;
+
+  public BukkitMetricsAdapter(final OriginBlacklistBukkit plugin) {
+    super();
+    this.plugin = plugin;
+  }
+
+  @Override
+  public void start() {
+    if (this.metrics == null) {
+      this.metrics = new Metrics(this.plugin, OriginBlacklist.BSTATS.BUKKIT);
+      for (final CustomChart chart : this.charts) {
+        this.metrics.addCustomChart(chart);
+      }
+    }
+  }
+
+  @Override
+  public void shutdown() {
+    if (this.metrics != null) {
+      this.metrics.shutdown();
+      this.metrics = null;
+    }
+  }
+}

src/main/java/xyz/webmc/originblacklist/bukkit/OriginBlacklistBukkit.java

@@ -136,6 +136,11 @@ public final class OriginBlacklistBukkit extends JavaPlugin implements Listener,
     return Paths.get(this.getFile().getAbsolutePath());
   }
 
+  @Override
+  public final BukkitMetricsAdapter getMetrics() {
+    return new BukkitMetricsAdapter(this);
+  }
+
   @Override
   public final IEaglerXServerAPI getEaglerAPI() {
     return EaglerXServerAPI.instance();
@@ -248,7 +253,6 @@ public final class OriginBlacklistBukkit extends JavaPlugin implements Listener,
 
   @Override
   public final void shutdown() {
-    this.metrics.shutdown();
     Bukkit.getScheduler().cancelTasks(this);
   }
 }

src/main/java/xyz/webmc/originblacklist/bungee/BungeeMetricsAdapter.java

@@ -0,0 +1,35 @@
+package xyz.webmc.originblacklist.bungee;
+
+import xyz.webmc.originblacklist.base.OriginBlacklist;
+import xyz.webmc.originblacklist.base.metrics.GenericMetricsAdapter;
+
+import org.bstats.charts.CustomChart;
+import org.bstats.bungeecord.Metrics;
+
+public final class BungeeMetricsAdapter extends GenericMetricsAdapter {
+  private final OriginBlacklistBungee plugin;
+  private Metrics metrics;
+
+  public BungeeMetricsAdapter(final OriginBlacklistBungee plugin) {
+    super();
+    this.plugin = plugin;
+  }
+
+  @Override
+  public void start() {
+    if (this.metrics == null) {
+      this.metrics = new Metrics(this.plugin, OriginBlacklist.BSTATS.BUNGEE);
+      for (final CustomChart chart : this.charts) {
+        this.metrics.addCustomChart(chart);
+      }
+    }
+  }
+
+  @Override
+  public void shutdown() {
+    if (this.metrics != null) {
+      this.metrics.shutdown();
+      this.metrics = null;
+    }
+  }
+}

src/main/java/xyz/webmc/originblacklist/bungee/OriginBlacklistBungee.java

@@ -14,8 +14,6 @@ import xyz.webmc.originblacklist.bungee.command.OriginBlacklistCommandBungee;
 import java.net.InetSocketAddress;
 import java.nio.file.Path;
 import java.nio.file.Paths;
-import java.util.HashMap;
-import java.util.Map;
 import java.util.concurrent.TimeUnit;
 
 import net.kyori.adventure.text.Component;
@@ -34,8 +32,6 @@ import net.md_5.bungee.api.plugin.Listener;
 import net.md_5.bungee.api.plugin.Plugin;
 import net.md_5.bungee.event.EventHandler;
 import net.md_5.bungee.event.EventPriority;
-import org.bstats.bungeecord.Metrics;
-import org.bstats.charts.AdvancedPie;
 import org.semver4j.Semver;
 
 @SuppressWarnings({ "deprecation", "rawtypes" })
@@ -44,7 +40,6 @@ public final class OriginBlacklistBungee extends Plugin implements Listener, IOr
   private boolean papiPlaceholdersEnabled;
   private Object papi;
   private OriginBlacklist blacklist;
-  private Metrics metrics;
 
   @Override
   public final void onEnable() {
@@ -75,20 +70,6 @@ public final class OriginBlacklistBungee extends Plugin implements Listener, IOr
         new OriginBlacklistCommandBungee(this, this.blacklist, "originblacklist"));
     this.getProxy().getPluginManager().registerListener(this, this);
     this.blacklist.init();
-    if (this.blacklist.isMetricsEnabled()) {
-      this.metrics = new Metrics(this, OriginBlacklist.BSTATS.BUNGEE);
-      this.metrics.addCustomChart(new AdvancedPie("player_types", () -> {
-        final Map<String, Integer> playerMap = new HashMap<>();
-
-        for (final ProxiedPlayer player : this.proxy.getPlayers()) {
-          final boolean eagler = this.getEaglerAPI().isEaglerPlayerByUUID(player.getUniqueId());
-          final String key = eagler ? "Eagler" : "Java";
-          playerMap.put(key, playerMap.getOrDefault(key, 0) + 1);
-        }
-
-        return playerMap;
-      }));
-    }
   }
 
   @Override
@@ -152,6 +133,11 @@ public final class OriginBlacklistBungee extends Plugin implements Listener, IOr
     return Paths.get(this.getFile().getAbsolutePath());
   }
 
+  @Override
+  public final BungeeMetricsAdapter getMetrics() {
+    return new BungeeMetricsAdapter(this);
+  }
+
   @Override
   public final IEaglerXServerAPI getEaglerAPI() {
     return EaglerXServerAPI.instance();
@@ -226,7 +212,6 @@ public final class OriginBlacklistBungee extends Plugin implements Listener, IOr
 
   @Override
   public final void shutdown() {
-    this.metrics.shutdown();
     this.proxy.getScheduler().cancel(this);
   }
 }

src/main/java/xyz/webmc/originblacklist/velocity/OriginBlacklistVelocity.java

@@ -14,9 +14,7 @@ import xyz.webmc.originblacklist.velocity.command.OriginBlacklistCommandVelocity
 import java.net.InetSocketAddress;
 import java.nio.file.Path;
 import java.nio.file.Paths;
-import java.util.HashMap;
 import java.util.List;
-import java.util.Map;
 import java.util.concurrent.TimeUnit;
 
 import com.google.inject.Inject;
@@ -39,13 +37,11 @@ import net.lax1dude.eaglercraft.backend.server.api.IEaglerXServerAPI;
 import net.lax1dude.eaglercraft.backend.server.api.velocity.EaglerXServerAPI;
 import net.lax1dude.eaglercraft.backend.server.api.velocity.event.EaglercraftLoginEvent;
 import net.lax1dude.eaglercraft.backend.server.api.velocity.event.EaglercraftMOTDEvent;
-import org.bstats.charts.AdvancedPie;
-import org.bstats.velocity.Metrics;
 import org.bstats.velocity.Metrics.Factory;
 import org.semver4j.Semver;
 import org.slf4j.Logger;
 
-@SuppressWarnings({ "deprecation", "unchecked", "rawtypes" })
+@SuppressWarnings({ "deprecation", "rawtypes" })
 public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
   private final PluginContainer plugin;
   private final Factory metricsFactory;
@@ -55,7 +51,6 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
   private boolean papiPlaceholdersEnabled;
   private Object papi;
   private OriginBlacklist blacklist;
-  private Metrics metrics;
 
   @Inject
   public OriginBlacklistVelocity(final PluginContainer plugin, Factory metricsFactory, final ProxyServer proxy,
@@ -67,7 +62,7 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
   }
 
   @Subscribe
-  public final void onProxyInitialization(ProxyInitializeEvent event) {
+  public final void onProxyInitialization(final ProxyInitializeEvent event) {
     this.proxy.getPluginManager().getPlugin("eaglerxserver").ifPresentOrElse(plugin -> {
       final Semver version = new Semver(plugin.getDescription().getVersion().orElse("1.0.0"));
       if (version.isLowerThan(OriginBlacklist.REQUIRED_API_VER)) {
@@ -91,20 +86,6 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
     this.blacklist = new OriginBlacklist(this);
     this.proxy.getCommandManager().register("originblacklist", new OriginBlacklistCommandVelocity(this.blacklist));
     this.blacklist.init();
-    if (this.blacklist.isMetricsEnabled()) {
-      this.metrics = this.metricsFactory.make(this, OriginBlacklist.BSTATS.VELOCITY);
-      this.metrics.addCustomChart(new AdvancedPie("player_types", () -> {
-        final Map<String, Integer> playerMap = new HashMap<>();
-
-        for (final Player player : this.proxy.getAllPlayers()) {
-          final boolean eagler = this.getEaglerAPI().isEaglerPlayerByUUID(player.getUniqueId());
-          final String key = eagler ? "Eagler" : "Java";
-          playerMap.put(key, playerMap.getOrDefault(key, 0) + 1);
-        }
-
-        return playerMap;
-      }));
-    }
   }
 
   @Subscribe
@@ -173,6 +154,11 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
     }
   }
 
+  @Override
+  public final VelocityMetricsAdapter getMetrics() {
+    return new VelocityMetricsAdapter(this, this.metricsFactory);
+  }
+
   @Override
   public final IEaglerXServerAPI getEaglerAPI() {
     return EaglerXServerAPI.instance();
@@ -254,7 +240,6 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
 
   @Override
   public final void shutdown() {
-    this.metrics.shutdown();
     for (ScheduledTask task : this.proxy.getScheduler().tasksByPlugin(this.plugin)) {
       task.cancel();
     }

src/main/java/xyz/webmc/originblacklist/velocity/VelocityMetricsAdapter.java

@@ -0,0 +1,38 @@
+package xyz.webmc.originblacklist.velocity;
+
+import xyz.webmc.originblacklist.base.OriginBlacklist;
+import xyz.webmc.originblacklist.base.metrics.GenericMetricsAdapter;
+
+import org.bstats.charts.CustomChart;
+import org.bstats.velocity.Metrics;
+import org.bstats.velocity.Metrics.Factory;
+
+public final class VelocityMetricsAdapter extends GenericMetricsAdapter {
+  private final OriginBlacklistVelocity plugin;
+  private final Factory factory;
+  private Metrics metrics;
+
+  public VelocityMetricsAdapter(final OriginBlacklistVelocity plugin, final Factory factory) {
+    super();
+    this.plugin = plugin;
+    this.factory = factory;
+  }
+
+  @Override
+  public void start() {
+    if (this.metrics == null) {
+      this.metrics = this.factory.make(this.plugin, OriginBlacklist.BSTATS.VELOCITY);
+      for (final CustomChart chart : this.charts) {
+        this.metrics.addCustomChart(chart);
+      }
+    }
+  }
+
+  @Override
+  public void shutdown() {
+    if (this.metrics != null) {
+      this.metrics.shutdown();
+      this.metrics = null;
+    }
+  }
+}