update readme

README.md

@@ -14,9 +14,9 @@
 - [x] MiniMessage and legacy formattings supported
 - [x] Plugin update checker
 - [x] Send blacklist logs to a webhook
-- [ ] Ingame blacklist management command
+- [x] Ingame blacklist management commands
+- [x] Reverse blacklist (whitelist)
 - [ ] Subscribe to an auto-updating blacklist
-- [ ] Reverse blacklist (whitelist)
 
 <h2>Changes from v1</h2>
 

build.gradle.kts

@@ -13,7 +13,7 @@ val PLUGIN_NAME = "OriginBlacklist"
 val PLUGIN_IDEN = "originblacklist"
 val PLUGIN_DOMN = "xyz.webmc"
 val PLUGIN_DESC = "An eaglercraft client blacklist plugin."
-val PLUGIN_VERS = "2.0.4"
+val PLUGIN_VERS = "2.0.5"
 val PLUGIN_SITE = "https://github.com/WebMCDevelopment/$PLUGIN_IDEN"
 val PLUGIN_DEPA = listOf("EaglercraftXServer")
 val PLUGIN_DEPB = listOf("EaglercraftXServer")

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

@@ -24,7 +24,6 @@ import java.nio.file.StandardCopyOption;
 import java.util.ArrayList;
 import java.util.Base64;
 import java.util.List;
-import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.TimeUnit;
 
 import de.marhali.json5.Json5Array;
@@ -54,8 +53,8 @@ public final class OriginBlacklist {
     this.plugin = plugin;
     this.config = new OriginBlacklistConfig(plugin);
     plugin.scheduleRepeat(() -> {
-      this.checkForUpdate();
-    }, 60, TimeUnit.MINUTES);
+      this.checkForUpdates();
+    }, this.config.getInteger("update_checker.check_timer"), TimeUnit.SECONDS);
   }
 
   public final void init() {
@@ -109,11 +108,11 @@ public final class OriginBlacklist {
   }
 
   public final boolean isDebugEnabled() {
-    return this.config.get("debug").getAsBoolean();
+    return this.config.getBoolean("debug");
   }
 
   public final boolean isMetricsEnabled() {
-    return this.config.get("bStats").getAsBoolean();
+    return this.config.getBoolean("bStats");
   }
 
   public final OriginBlacklistConfig getConfig() {
@@ -135,57 +134,132 @@ public final class OriginBlacklist {
     conn.disconnect();
   }
 
-  private final EnumBlacklistType testBlacklist(final OPlayer player) {
+  public final void checkForUpdates(Runnable action1, Runnable action2) {
+    if (this.config.getBoolean("update_checker.enabled")) {
+      this.plugin.runAsync(() -> {
+        this.updateURL = UpdateChecker.checkForUpdates(PLUGIN_REPO, this.plugin.getPluginVersion(),
+            this.config.getBoolean("update_checker.allow_snapshots"));
+        if (isNonNull((this.updateURL))) {
+          action1.run();
+          return;
+        }
+        action2.run();
+      });
+    } else {
+      action2.run();
+    }
+  }
+
+  public final void updatePlugin(Runnable action1, Runnable action2) {
+    try {
+      final URL url = new URL(this.updateURL);
+      final Path jar = this.plugin.getPluginJarPath();
+      final Path bak = jar.resolveSibling(jar.getFileName().toString() + ".bak");
+      final Path upd = jar.resolveSibling(Paths.get(URLDecoder.decode(url.getPath(), StandardCharsets.UTF_8)).getFileName());
+
+      try {
+        Files.copy(jar, bak, StandardCopyOption.REPLACE_EXISTING);
+      } catch (final Throwable t) {
+        t.printStackTrace();
+      }
+      
+      try {
+        final HttpURLConnection conn = (HttpURLConnection) url.openConnection();
+        conn.setRequestMethod("GET");
+        conn.setConnectTimeout(15000);
+        conn.setReadTimeout(15000);
+        conn.setRequestProperty("User-Agent", OriginBlacklist.getUserAgent());
+        conn.connect();
+        try (final InputStream in = conn.getInputStream()) {
+          Files.copy(in, upd, StandardCopyOption.REPLACE_EXISTING);
+        } finally {
+          conn.disconnect();
+        }
+        Files.delete(jar);
+        Files.delete(bak);
+        action1.run();
+        return;
+      } catch (final Throwable t) {
+        t.printStackTrace();
+        Files.move(bak, jar, StandardCopyOption.REPLACE_EXISTING);
+      }
+    } catch (final Throwable t) {
+      t.printStackTrace();
+    }
+    action2.run();
+  }
+
+  public final void updatePlugin() {
+    this.updatePlugin(() -> {}, () -> {});
+  }
+
+  public final EnumBlacklistType testBlacklist(final OPlayer player) {
     final String name = player.getName();
     final String addr = player.getAddr();
     final String origin = player.getOrigin();
     final String brand = player.getBrand();
 
+    final boolean whitelist = this.config.getBoolean("blacklist_to_whitelist");
+    EnumBlacklistType type = EnumBlacklistType.NONE;
+
     if (isNonNull(origin)) {
-      for (final Json5Element element : this.config.get("blacklist.origins").getAsJson5Array()) {
+      if (whitelist && !type.isBlacklisted()) type = EnumBlacklistType.ORIGIN;
+      for (final Json5Element element : this.config.getArray("blacklist.origins").getAsJson5Array()) {
         if (origin.matches(element.getAsString())) {
-          return EnumBlacklistType.ORIGIN;
+          if (whitelist) type = EnumBlacklistType.NONE;
+          else if (!type.isBlacklisted()) type = EnumBlacklistType.ORIGIN;
+          break;
         }
       }
+    } else if (this.config.getBoolean("block_undefined_origin")) {
+      return whitelist ? EnumBlacklistType.NONE : EnumBlacklistType.ORIGIN;
     }
 
     if (isNonNull(brand)) {
-      for (final Json5Element element : this.config.get("blacklist.brands").getAsJson5Array()) {
+      if (whitelist && !type.isBlacklisted()) type = EnumBlacklistType.BRAND;
+      for (final Json5Element element : this.config.getArray("blacklist.brands")) {
         if (brand.matches(element.getAsString())) {
-          return EnumBlacklistType.BRAND;
+          if (whitelist) type = EnumBlacklistType.NONE;
+          else if (!type.isBlacklisted()) type = EnumBlacklistType.BRAND;
+          break;
         }
       }
     }
 
     if (isNonNull(name)) {
-      for (final Json5Element element : this.config.get("blacklist.player_names").getAsJson5Array()) {
-        this.plugin.log(EnumLogLevel.DEBUG, element.getAsString());
+      if (whitelist && !type.isBlacklisted()) type = EnumBlacklistType.NAME;
+      for (final Json5Element element : this.config.getArray("blacklist.player_names")) {
         if (name.matches(element.getAsString())) {
-          return EnumBlacklistType.NAME;
+          if (whitelist) type = EnumBlacklistType.NONE;
+          else if (!type.isBlacklisted()) type = EnumBlacklistType.NAME;
+          break;
         }
       }
     }
 
     if (isNonNull(addr)) {
-      for (final Json5Element element : this.config.get("blacklist.ip_addresses").getAsJson5Array()) {
+      if (whitelist && !type.isBlacklisted()) type = EnumBlacklistType.ADDR;
+      for (final Json5Element element : this.config.getArray("blacklist.ip_addresses")) {
         try {
           if ((new IPAddressString(element.getAsString()).toAddress())
               .contains((new IPAddressString(addr)).toAddress())) {
-            return EnumBlacklistType.ADDR;
+            if (whitelist) type = EnumBlacklistType.NONE;
+            else if (!type.isBlacklisted()) type = EnumBlacklistType.ADDR;
+            break;
           }
         } catch (final AddressStringException exception) {
-          // exception.printStackTrace();
+          if (this.isDebugEnabled()) exception.printStackTrace();
         }
       }
     }
 
-    return EnumBlacklistType.NONE;
+    return type;
   }
 
   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 Json5Array arr = this.config.get("messages." + type).getAsJson5Array();
+    final Json5Array arr = this.config.getArray("messages." + type);
     final StringBuilder sb = new StringBuilder();
     for (int i = 0; i < arr.size(); i++) {
       if (i > 0)
@@ -193,7 +267,7 @@ public final class OriginBlacklist {
       sb.append(arr.get(i).getAsString());
     }
     final String str = sb.toString()
-        .replaceAll("%action%", this.config.get("messages.actions." + action).getAsString())
+        .replaceAll("%action%", this.config.getString("messages.actions." + action))
         .replaceAll("%block_type%", blockType)
         .replaceAll("%block_type%", blockType)
         .replaceAll("%not_allowed%", notAllowed)
@@ -203,8 +277,7 @@ public final class OriginBlacklist {
   }
 
   private final void sendWebhooks(final OriginBlacklistLoginEvent event, final EnumBlacklistType type) {
-    Json5Element element = this.config.get("discord.enabled");
-    if (element.getAsBoolean()) {
+    if (this.config.getBoolean("discord.enabled")) {
       final OPlayer player = event.getPlayer();
       final EnumConnectionType connType = event.getConnectionType();
       final String userAgent;
@@ -252,7 +325,7 @@ public final class OriginBlacklist {
         player.getName().replaceAll("_", "\\_"),
         player.getOrigin(),
         player.getBrand(),
-        this.config.get("discord.send_ips").getAsBoolean() ? player.getAddr() : "*\\*CENSORED\\**",
+        this.config.getBoolean("discord.send_ips") ? player.getAddr() : "*\\*CENSORED\\**",
         player.getPVN(),
         userAgent,
         player.isRewind() ? "YES" : "NO",
@@ -261,88 +334,44 @@ public final class OriginBlacklist {
         PLUGIN_REPO,
         PLUGIN_REPO
       ).getBytes();
-      element = this.config.get("discord.webhook_urls");
-      if (element instanceof Json5Array) {
-        for (final Json5Element _element : element.getAsJson5Array()) {
-          CompletableFuture.runAsync(() -> {
-            try {
-              final URL url = new URL(_element.getAsString());
-              final HttpURLConnection conn = (HttpURLConnection) url.openConnection();
-              conn.setRequestMethod("POST");
-              conn.setRequestProperty("Content-Type", "application/json");
-              conn.setDoOutput(true);
-              conn.setConnectTimeout(5000);
-              conn.setReadTimeout(5000);
-              conn.connect();
-              final OutputStream os = conn.getOutputStream();
-              os.write(payload);
-              os.close();
+      final Json5Array arr = this.config.get("discord.webhook_urls").getAsJson5Array();
+      for (final Json5Element element : arr) {
+        this.plugin.runAsync(() -> {
+          try {
+            final URL url = new URL(element.getAsString());
+            final HttpURLConnection conn = (HttpURLConnection) url.openConnection();
+            conn.setRequestMethod("POST");
+            conn.setRequestProperty("Content-Type", "application/json");
+            conn.setDoOutput(true);
+            conn.setConnectTimeout(5000);
+            conn.setReadTimeout(5000);
+            conn.connect();
+            final OutputStream os = conn.getOutputStream();
+            os.write(payload);
+            os.close();
 
-              final int code = conn.getResponseCode();
-              if (code < 200 || code >= 300) {
-                this.plugin.log(EnumLogLevel.WARN, "Webhook failed (HTTP " + code + ")");
-              }
-
-              conn.disconnect();
-            } catch (final Throwable t) {
-              t.printStackTrace();
+            final int code = conn.getResponseCode();
+            if (code < 200 || code >= 300) {
+              this.plugin.log(EnumLogLevel.WARN, "Webhook failed (HTTP " + code + ")");
             }
-          });
-        }
-      }
-    }
-  }
 
-  private final void checkForUpdate() {
-    if (this.config.get("update_checker.enabled").getAsBoolean()) {
-      CompletableFuture.runAsync(() -> {
-        this.updateURL = UpdateChecker.checkForUpdate(PLUGIN_REPO, this.plugin.getPluginVersion(),
-            this.config.get("update_checker.allow_snapshots").getAsBoolean());
-        if (isNonNull((this.updateURL))) {
-          if (!this.config.get("update_checker.auto_update").getAsBoolean()) {
-            this.plugin.log(EnumLogLevel.INFO, "An update is available! Download it at " + this.updateURL);
-          } else {
-            this.updatePlugin();
+            conn.disconnect();
+          } catch (final Throwable t) {
+            t.printStackTrace();
           }
-        }
-      });
+        });
+      }
     }
   }
 
-  private final void updatePlugin() {
-    try {
-      final URL url = new URL(this.updateURL);
-      final Path jar = this.plugin.getPluginJarPath();
-      final Path bak = jar.resolveSibling(jar.getFileName().toString() + ".bak");
-      final Path upd = jar.resolveSibling(Paths.get(URLDecoder.decode(url.getPath(), StandardCharsets.UTF_8)).getFileName());
-
-      try {
-        Files.copy(jar, bak, StandardCopyOption.REPLACE_EXISTING);
-      } catch (final Throwable t) {
-        t.printStackTrace();
+  private final void checkForUpdates() {
+    this.checkForUpdates(() -> {
+      if (!this.config.getBoolean("update_checker.auto_update")) {
+        this.plugin.log(EnumLogLevel.INFO, "An update is available! Download it at " + this.updateURL);
+      } else {
+        this.updatePlugin();
       }
-      
-      try {
-        final HttpURLConnection conn = (HttpURLConnection) url.openConnection();
-        conn.setRequestMethod("GET");
-        conn.setConnectTimeout(15000);
-        conn.setReadTimeout(15000);
-        conn.setRequestProperty("User-Agent", OriginBlacklist.getUserAgent());
-        conn.connect();
-        try (final InputStream in = conn.getInputStream()) {
-          Files.copy(in, upd, StandardCopyOption.REPLACE_EXISTING);
-        } finally {
-          conn.disconnect();
-        }
-        Files.delete(jar);
-        Files.delete(bak);
-      } catch (final Throwable t) {
-        t.printStackTrace();
-        Files.move(bak, jar, StandardCopyOption.REPLACE_EXISTING);
-      }
-    } catch (final Throwable t) {
-      t.printStackTrace();
-    }
+    }, () -> {});
   }
 
   public static final String getComponentString(final Component comp) {

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

@@ -1,8 +1,15 @@
 package xyz.webmc.originblacklist.base.command;
 
+import xyz.webmc.originblacklist.base.OriginBlacklist;
+
 public interface CommandContext {
+  public OriginBlacklist getPlugin();
+
   public String getPlayerName();
+
   public void reply(final String message);
+
   public boolean hasPermission(final String permission);
+
   public String[] getArgs();
 }

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

@@ -4,7 +4,10 @@ import java.util.List;
 
 public interface ICommand {
   public static final String NO_PERMISSION = "<red>You don't have permission to use this command.</red>";
+
   public boolean execute(final CommandContext ctx);
+
   public List<String> suggest(final CommandContext ctx);
+
   public void usage(final CommandContext ctx);
 }

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

@@ -1,10 +1,15 @@
 package xyz.webmc.originblacklist.base.command;
 
 import xyz.webmc.originblacklist.base.OriginBlacklist;
+import xyz.webmc.originblacklist.base.config.OriginBlacklistConfig;
+import xyz.webmc.originblacklist.base.enums.EnumBlacklistType;
+import xyz.webmc.originblacklist.base.util.OPlayer;
 
 import java.util.List;
 
+import de.marhali.json5.Json5Array;
 import de.marhali.json5.Json5Element;
+import de.marhali.json5.Json5Primitive;
 
 public class OriginBlacklistCommand implements ICommand {
   private final OriginBlacklist plugin;
@@ -18,31 +23,104 @@ public class OriginBlacklistCommand implements ICommand {
     final String[] args = ctx.getArgs();
     if (ctx.hasPermission("originblacklist.command")) {
       if (args.length > 0) {
+        final OriginBlacklistConfig config = this.plugin.getConfig();
         final String command = args[0].toLowerCase();
+        final String argA = args.length > 1 ? args[1].toLowerCase() : null;
+        ;
+        final String argB = args.length > 2 ? args[2] : null;
+        final boolean add = "add".equals(command);
+        final boolean remove = "remove".equals(command);
         if ("reload".equals(command)) {
           if (ctx.hasPermission("originblacklist.command.reload")) {
-            this.plugin.getConfig().reloadConfig();
+            config.reloadConfig();
             ctx.reply("<green>Configuration Reloaded</green>");
           } else {
             ctx.reply(NO_PERMISSION);
           }
+        } else if ("update".equals(command)) {
+          if (ctx.hasPermission("originblacklist.command.update")) {
+            ctx.reply("<aqua>Checking for updates...</aqua>");
+            this.plugin.checkForUpdates(() -> {
+              ctx.reply("<yellow>Updating plugin...</yellow>");
+              this.plugin.updatePlugin(() -> {
+                ctx.reply("<green>Successfully updated plugin.</green>");
+              }, () -> {
+                ctx.reply("<red>Failed to update plugin.</red>");
+              });
+            }, () -> {
+              ctx.reply("<green>Plugin is up to date.</green>");
+            });
+          } else {
+            ctx.reply(NO_PERMISSION);
+          }
+        } else if ((add || remove) && OriginBlacklist.isNonNull(argB)) {
+          if ((add && ctx.hasPermission("originblacklist.command.add"))
+              || (remove && ctx.hasPermission("originblacklist.command.add"))) {
+            final String arrName;
+            if ("origin".equals(argA)) {
+              arrName = "origins";
+            } else if ("brand".equals(argA)) {
+              arrName = "brands";
+            } else if ("name".equals(argA)) {
+              arrName = "player_names";
+            } else if ("ip".equals(argA)) {
+              arrName = "ip_addresses";
+            } else {
+              arrName = null;
+            }
+            if (OriginBlacklist.isNonNull(arrName)) {
+              final String arrPath = "blacklist." + arrName;
+              final Json5Array arr = config.getArray(arrPath);
+              if (add) {
+                if (!arr.contains(Json5Primitive.fromString(argB))) {
+                  arr.add(argB);
+                  config.set(arrPath, arr);
+                  ctx.reply("<green>Added " + argB + " to the " + argA + " blacklist</green>");
+                } else {
+                  ctx.reply("<red>" + argB + " is already on the " + argA + " blacklist</red>");
+                }
+              } else if (remove) {
+                if (arr.contains(Json5Primitive.fromString(argB))) {
+                  arr.remove(Json5Primitive.fromString(argB));
+                  config.set(arrPath, arr);
+                  ctx.reply("<green>Removed " + argB + " from the " + argA + " blacklist</green>");
+                } else {
+                  ctx.reply("<red>" + argB + " not on the " + argA + " blacklist</red>");
+                }
+              }
+            } else {
+              this.usage(ctx);
+            }
+          } else {
+            ctx.reply(NO_PERMISSION);
+          }
+        } else if ("test".equals(command) && OriginBlacklist.isNonNull(argA)) {
+          if (ctx.hasPermission("originblacklist.command.test")) {
+            if (this.isBlacklisted(argA)) {
+              ctx.reply("<green>" + argA + " is on the blacklist.</green>");
+            } else {
+              ctx.reply("<red>" + argA + " is not on the blacklist.</red>");
+            }
+          } else {
+            ctx.reply(NO_PERMISSION);
+          }
         } else if ("list".equals(command)) {
-          if (ctx.hasPermission("originblacklist.command.reload")) {
+          if (ctx.hasPermission("originblacklist.command.list")) {
             ctx.reply("<aqua>Blacklist:</aqua>");
             ctx.reply("<gold>  - Origins:</gold>");
-            for (final Json5Element element : this.plugin.getConfig().get("blacklist.origins").getAsJson5Array()) {
+            for (final Json5Element element : config.getArray("blacklist.origins")) {
               ctx.reply("<gray>    - " + element.getAsString() + "</gray>");
             }
             ctx.reply("<gold>  - Brands:</gold>");
-            for (final Json5Element element : this.plugin.getConfig().get("blacklist.brands").getAsJson5Array()) {
+            for (final Json5Element element : config.getArray("blacklist.brands")) {
               ctx.reply("<gray>    - " + element.getAsString() + "</gray>");
             }
             ctx.reply("<gold>  - Players:</gold>");
-            for (final Json5Element element : this.plugin.getConfig().get("blacklist.player_names").getAsJson5Array()) {
+            for (final Json5Element element : config.getArray("blacklist.player_names")) {
               ctx.reply("<gray>    - " + element.getAsString() + "</gray>");
             }
             ctx.reply("<gold>  - IPs:</gold>");
-            for (final Json5Element element : this.plugin.getConfig().get("blacklist.ip_addresses").getAsJson5Array()) {
+            for (final Json5Element element : config.getArray("blacklist.ip_addresses")) {
               ctx.reply("<gray>    - " + element.getAsString() + "</gray>");
             }
           } else {
@@ -61,16 +139,23 @@ public class OriginBlacklistCommand implements ICommand {
   }
 
   @Override
-  public List<String> suggest(final CommandContext ctx) {
+  public final List<String> suggest(final CommandContext ctx) {
     return List.of();
   }
 
   @Override
-  public void usage(CommandContext ctx) {
+  public final void usage(CommandContext ctx) {
     ctx.reply("<aqua>Commands:</aqua>");
     ctx.reply("<gray>  - /originblacklist reload</gray>");
-    // ctx.reply("<gray>  - /originblacklist add <brand/origin/name/ip> <value></gray>");
-    // ctx.reply("<gray>  - /originblacklist remove <brand/origin/name/ip> <value></gray>");
+    ctx.reply("<gray>  - /originblacklist update</gray>");
+    ctx.reply("<gray>  - /originblacklist add <origin/brand/name/ip> <argB></gray>");
+    ctx.reply("<gray>  - /originblacklist remove <origin/brand/name/ip> <argB></gray>");
+    ctx.reply("<gray>  - /originblacklist test <argB></gray>");
     ctx.reply("<gray>  - /originblacklist list</gray>");
   }
+
+  private final boolean isBlacklisted(final String str) {
+    final OPlayer player = new OPlayer(null, str, null, str, str, -1);
+    return this.plugin.testBlacklist(player) != EnumBlacklistType.NONE;
+  }
 }

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

@@ -31,7 +31,7 @@ public final class OriginBlacklistConfig {
   private byte[] icon;
   private String icon64;
 
-  public OriginBlacklistConfig(IOriginBlacklistPlugin plugin) {
+  public OriginBlacklistConfig(final IOriginBlacklistPlugin plugin) {
     this.json5 = Json5.builder(builder -> builder
         .quoteless()
         .quoteSingle()
@@ -132,7 +132,7 @@ public final class OriginBlacklistConfig {
 
     if (this.config != null && OriginBlacklist.isNonNull(key)) {
       element = this.config;
-      final String[] parts = key.split("\\.");
+      final String[] parts = splitPath(key);
 
       for (final String part : parts) {
         if (element instanceof Json5Object) {
@@ -155,6 +155,95 @@ public final class OriginBlacklistConfig {
     return element;
   }
 
+  public final boolean set(final String key, final Json5Element value) {
+    boolean ret = false;
+
+    if (this.config != null && value != null) {
+      final String[] parts = splitPath(key);
+
+      if (parts.length > 0) {
+        Json5Object obj = this.config;
+
+        for (int i = 0; i < parts.length - 1; i++) {
+          final String part = parts[i];
+          final Json5Element cur = obj.has(part) ? obj.get(part) : null;
+
+          if (cur instanceof Json5Object next) {
+            obj = next;
+          } else {
+            final Json5Object next = new Json5Object();
+            obj.add(part, next);
+            obj = next;
+          }
+        }
+
+        obj.add(parts[parts.length - 1], value.deepCopy());
+        this.saveConfig();
+        ret = true;
+      }
+    }
+
+    return ret;
+  }
+
+  public final boolean remove(final String key) {
+    boolean ret = false;
+
+    if (this.config != null) {
+      final String[] parts = splitPath(key);
+
+      if (parts.length > 0) {
+        Json5Object obj = this.config;
+        Json5Element element = obj;
+
+        for (int i = 0; i < parts.length - 1; i++) {
+          if (element instanceof Json5Object cur && cur.has(parts[i])) {
+            element = cur.get(parts[i]);
+            if (element instanceof Json5Object) {
+              obj = (Json5Object) element;
+            } else {
+              element = null;
+            }
+          } else {
+            element = null;
+          }
+
+          if (element == null) {
+            break;
+          }
+        }
+
+        if (element != null && obj.has(parts[parts.length - 1])) {
+          obj.remove(parts[parts.length - 1]);
+          this.saveConfig();
+          ret = true;
+        }
+      }
+    }
+
+    return ret;
+  }
+
+  public final String getString(final String key) {
+    return this.get(key).getAsString();
+  }
+
+  public final boolean getBoolean(final String key) {
+    return this.get(key).getAsBoolean();
+  }
+
+  public final int getInteger(final String key) {
+    return this.get(key).getAsInt();
+  }
+
+  public final Json5Array getArray(final String key) {
+    return this.get(key).getAsJson5Array();
+  }
+
+  public final Json5Object getObject(final String key) {
+    return this.get(key).getAsJson5Object();
+  }
+
   public final byte[] getIconBytes() {
     return this.icon;
   }
@@ -221,9 +310,13 @@ public final class OriginBlacklistConfig {
     final Json5Object uobj = new Json5Object();
     addJSONObj(uobj, "enabled", Json5Primitive.fromBoolean(true), null);
     addJSONObj(uobj, "allow_snapshots", Json5Primitive.fromBoolean(false), null);
+    addJSONObj(uobj, "check_timer", Json5Primitive.fromNumber(3600), null);
     addJSONObj(uobj, "auto_update", Json5Primitive.fromBoolean(true), null);
     addJSONObj(obj, "update_checker", uobj, null);
+    addJSONObj(obj, "blacklist_to_whitelist", Json5Primitive.fromBoolean(false), null);
+    addJSONObj(obj, "block_undefined_origin", Json5Primitive.fromBoolean(false), null);
     addJSONObj(obj, "bStats", Json5Primitive.fromBoolean(true), null);
+    addJSONObj(obj, "config_version", Json5Primitive.fromNumber(1), "DO NOT CHANGE");
     return obj;
   }
 
@@ -255,4 +348,16 @@ public final class OriginBlacklistConfig {
     }
     obj.add(key, value);
   }
+
+  private static final String[] splitPath(final String key) {
+    final String[] ret;
+
+    if (OriginBlacklist.isNonNull(key)) {
+      ret = key.split("\\.");
+    } else {
+      ret = new String[0];
+    }
+
+    return ret;
+  }
 }

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

@@ -36,4 +36,8 @@ public enum EnumBlacklistType {
   public final String getActionString() {
     return this.act;
   }
+
+  public final boolean isBlacklisted() {
+    return this != NONE;
+  }
 }

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

@@ -3,7 +3,7 @@ package xyz.webmc.originblacklist.base.util;
 import java.io.InputStream;
 import java.util.Properties;
 
-public class BuildInfo {
+public final class BuildInfo {
   private static final Properties properties;
 
   public static final String get(final String key) {
@@ -14,6 +14,7 @@ public class BuildInfo {
     properties = new Properties();
     try (final InputStream in = BuildInfo.class.getClassLoader().getResourceAsStream("build.properties")) {
       properties.load(in);
-    } catch (final Throwable t) {}
+    } catch (final Throwable t) {
+    }
   }
 }

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

@@ -12,12 +12,22 @@ import org.semver4j.Semver;
 
 public interface IOriginBlacklistPlugin {
   public String getPluginId();
+
   public Semver getPluginVersion();
+
   public Path getPluginJarPath();
+
   public void log(final EnumLogLevel level, final String txt);
+
   public void kickPlayer(final Component txt, final OriginBlacklistLoginEvent event);
+
   public void setMOTD(final Component txt, final OriginBlacklistMOTDEvent event);
+
   public String parsePlaceholders(final OPlayer player, final String str);
+
   public void scheduleRepeat(final Runnable task, final int period, final TimeUnit unit);
+
+  public void runAsync(final Runnable task);
+
   public void shutdown();
 }

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

@@ -78,9 +78,7 @@ public final class OPlayer {
   }
 
   private static final String formatIPAddress(String addr) {
-    if (addr == null) {
-      addr = OriginBlacklist.UNKNOWN_STR;
-    } else {
+    if (OriginBlacklist.isNonNull(addr)) {
       if (addr.startsWith("/")) {
         addr = addr.substring(1);
       }
@@ -132,6 +130,8 @@ public final class OPlayer {
       if (hex && c == 6 && addr.indexOf("::") == -1) {
         addr = addr + "::";
       }
+    } else {
+      addr = OriginBlacklist.UNKNOWN_STR;
     }
 
     return addr;

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

@@ -14,10 +14,11 @@ import de.marhali.json5.Json5Object;
 import org.semver4j.Semver;
 import org.semver4j.Semver.VersionDiff;
 
-public class UpdateChecker {
+public final class UpdateChecker {
   private static final Json5 json5 = Json5.builder(builder -> builder.build());
 
-  public static final String checkForUpdate(final String repo, final Semver currentVersion, final boolean allowSnapshots) {
+  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 HttpURLConnection conn = (HttpURLConnection) url.openConnection();
@@ -46,11 +47,13 @@ public class UpdateChecker {
             }
             final boolean pre = obj.get("prerelease").getAsBoolean();
             if (!pre) {
-              if (rel == null || obj.get("published_at").getAsString().compareTo(rel.get("published_at").getAsString()) > 0) {
+              if (rel == null
+                  || obj.get("published_at").getAsString().compareTo(rel.get("published_at").getAsString()) > 0) {
                 rel = obj;
               }
             } else {
-              if (snap == null || obj.get("published_at").getAsString().compareTo(snap.get("published_at").getAsString()) > 0) {
+              if (snap == null
+                  || obj.get("published_at").getAsString().compareTo(snap.get("published_at").getAsString()) > 0) {
                 snap = obj;
               }
             }
@@ -71,7 +74,8 @@ public class UpdateChecker {
           } catch (Throwable t) {
             comm = "";
           }
-          if (ver.isGreaterThan(currentVersion) || (allowSnapshots && currentVersion.diff(ver) == VersionDiff.BUILD && OriginBlacklist.isNonNull(comm) && !BuildInfo.get("git_cm_hash").startsWith(comm))) {
+          if (ver.isGreaterThan(currentVersion) || (allowSnapshots && currentVersion.diff(ver) == VersionDiff.BUILD
+              && OriginBlacklist.isNonNull(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/OriginBlacklistBukkit.java

@@ -234,6 +234,11 @@ public final class OriginBlacklistBukkit extends JavaPlugin implements Listener,
     Bukkit.getScheduler().runTaskTimer(this, task, 0, ticks);
   }
 
+  @Override
+  public final void runAsync(final Runnable task) {
+    Bukkit.getScheduler().runTaskAsynchronously(this, task);
+  }
+
   @Override
   public final void shutdown() {
     this.metrics.shutdown();

src/main/java/xyz/webmc/originblacklist/bukkit/command/BKTCommandContext.java

@@ -5,32 +5,39 @@ import xyz.webmc.originblacklist.base.command.CommandContext;
 
 import org.bukkit.command.CommandSender;
 
-public class BKTCommandContext implements CommandContext {
+public final class BKTCommandContext implements CommandContext {
+  private final OriginBlacklist plugin;
   private final CommandSender sender;
   private final String[] args;
 
-  public BKTCommandContext(final CommandSender sender, final String[] args) {
+  public BKTCommandContext(final OriginBlacklist plugin, final CommandSender sender, final String[] args) {
+    this.plugin = plugin;
     this.sender = sender;
     this.args = args;
   }
 
   @Override
-  public String getPlayerName() {
+  public final OriginBlacklist getPlugin() {
+    return this.plugin;
+  }
+
+  @Override
+  public final String getPlayerName() {
     return this.sender.getName();
   }
 
   @Override
-  public void reply(final String message) {
+  public final void reply(final String message) {
     this.sender.sendMessage(OriginBlacklist.getLegacyFromMiniMessage(message));
   }
 
   @Override
-  public boolean hasPermission(final String permission) {
+  public final boolean hasPermission(final String permission) {
     return this.sender.hasPermission(permission);
   }
 
   @Override
-  public String[] getArgs() {
+  public final String[] getArgs() {
     return this.args;
   }
 }

src/main/java/xyz/webmc/originblacklist/bukkit/command/OriginBlacklistCommandBukkit.java

@@ -9,18 +9,22 @@ import org.bukkit.command.Command;
 import org.bukkit.command.CommandSender;
 import org.bukkit.command.TabExecutor;
 
-public class OriginBlacklistCommandBukkit extends OriginBlacklistCommand implements TabExecutor {
+public final class OriginBlacklistCommandBukkit extends OriginBlacklistCommand implements TabExecutor {
+  private final OriginBlacklist plugin;
+
   public OriginBlacklistCommandBukkit(OriginBlacklist plugin) {
     super(plugin);
+    this.plugin = plugin;
   }
 
   @Override
   public boolean onCommand(final CommandSender sender, final Command command, final String label, final String[] args) {
-    return super.execute(new BKTCommandContext(sender, args));
+    return super.execute(new BKTCommandContext(this.plugin, sender, args));
   }
 
   @Override
-  public List<String> onTabComplete(final CommandSender sender, final Command command, final String label, final String[] args) {
-    return super.suggest(new BKTCommandContext(sender, args));
+  public List<String> onTabComplete(final CommandSender sender, final Command command, final String label,
+      final String[] args) {
+    return super.suggest(new BKTCommandContext(this.plugin, sender, args));
   }
 }

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

@@ -198,6 +198,11 @@ public final class OriginBlacklistBungee extends Plugin implements Listener, IOr
     this.proxy.getScheduler().schedule(this, task, 0, period, unit);
   }
 
+  @Override
+  public final void runAsync(final Runnable task) {
+    this.proxy.getScheduler().runAsync(this, task);
+  }
+
   @Override
   public final void shutdown() {
     this.metrics.shutdown();

src/main/java/xyz/webmc/originblacklist/bungee/command/BNGCommandContext.java

@@ -6,32 +6,39 @@ import xyz.webmc.originblacklist.base.command.CommandContext;
 import net.md_5.bungee.api.CommandSender;
 import net.md_5.bungee.api.chat.TextComponent;
 
-public class BNGCommandContext implements CommandContext {
+public final class BNGCommandContext implements CommandContext {
+  private final OriginBlacklist plugin;
   private final CommandSender sender;
   private final String[] args;
 
-  public BNGCommandContext(final CommandSender sender, final String[] args) {
+  public BNGCommandContext(final OriginBlacklist plugin, final CommandSender sender, final String[] args) {
+    this.plugin = plugin;
     this.sender = sender;
     this.args = args;
   }
 
   @Override
-  public String getPlayerName() {
+  public final OriginBlacklist getPlugin() {
+    return this.plugin;
+  }
+
+  @Override
+  public final String getPlayerName() {
     return this.sender.getName();
   }
 
   @Override
-  public void reply(final String message) {
+  public final void reply(final String message) {
     this.sender.sendMessage(TextComponent.fromLegacy(OriginBlacklist.getLegacyFromMiniMessage(message)));
   }
 
   @Override
-  public boolean hasPermission(final String permission) {
+  public final boolean hasPermission(final String permission) {
     return this.sender.hasPermission(permission);
   }
 
   @Override
-  public String[] getArgs() {
+  public final String[] getArgs() {
     return this.args;
   }
 }

src/main/java/xyz/webmc/originblacklist/bungee/command/OriginBlacklistCommandBungee.java

@@ -10,21 +10,24 @@ import net.md_5.bungee.api.CommandSender;
 import net.md_5.bungee.api.plugin.Command;
 import net.md_5.bungee.api.plugin.TabExecutor;
 
-public class OriginBlacklistCommandBungee extends Command implements TabExecutor {
+public final class OriginBlacklistCommandBungee extends Command implements TabExecutor {
   private final OriginBlacklistCommand cmd;
+  private final OriginBlacklist blacklist;
 
-  public OriginBlacklistCommandBungee(final OriginBlacklistBungee plugin, final OriginBlacklist blacklist, final String command) {
+  public OriginBlacklistCommandBungee(final OriginBlacklistBungee plugin, final OriginBlacklist blacklist,
+      final String command) {
     super(command);
     this.cmd = new OriginBlacklistCommand(blacklist);
+    this.blacklist = blacklist;
   }
 
   @Override
-  public void execute(final CommandSender sender, final String[] args) {
-    this.cmd.execute(new BNGCommandContext(sender, args));
+  public final void execute(final CommandSender sender, final String[] args) {
+    this.cmd.execute(new BNGCommandContext(this.blacklist, sender, args));
   }
 
   @Override
-  public List<String> onTabComplete(final CommandSender sender, final String[] args) {
-    return this.cmd.suggest(new BNGCommandContext(sender, args));
+  public final List<String> onTabComplete(final CommandSender sender, final String[] args) {
+    return this.cmd.suggest(new BNGCommandContext(this.blacklist, sender, args));
   }
 }

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

@@ -64,7 +64,7 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
   }
 
   @Subscribe
-  public void onProxyInitialization(ProxyInitializeEvent event) {
+  public final void onProxyInitialization(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)) {
@@ -120,7 +120,8 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
   @Subscribe(order = PostOrder.FIRST)
   public final void onJavaLogin(final PreLoginEvent event) {
     final OPlayer player = new OPlayer(null, event.getUsername(), event.getUniqueId(),
-        event.getConnection().getRemoteAddress().toString(), OriginBlacklist.UNKNOWN_STR, event.getConnection().getProtocolVersion().getProtocol());
+        event.getConnection().getRemoteAddress().toString(), OriginBlacklist.UNKNOWN_STR,
+        event.getConnection().getProtocolVersion().getProtocol());
     this.blacklist.handleLogin(new OriginBlacklistLoginEvent(null, event, EnumConnectionType.JAVA, player));
   }
 
@@ -128,7 +129,8 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
   public final void onJavaHandshake(final PlayerClientBrandEvent event) {
     final Player aPlayer = (Player) event.getPlayer();
     final OPlayer bPlayer = new OPlayer(null, aPlayer.getUsername(), aPlayer.getUniqueId(),
-        aPlayer.getRemoteAddress().getAddress().toString(), event.getBrand(), event.getPlayer().getProtocolVersion().getProtocol());
+        aPlayer.getRemoteAddress().getAddress().toString(), event.getBrand(),
+        event.getPlayer().getProtocolVersion().getProtocol());
     this.blacklist.handleLogin(new OriginBlacklistLoginEvent(null, event, EnumConnectionType.JAVA, bPlayer));
   }
 
@@ -225,6 +227,13 @@ public final class OriginBlacklistVelocity implements IOriginBlacklistPlugin {
         .schedule();
   }
 
+  @Override
+  public final void runAsync(final Runnable task) {
+    this.proxy.getScheduler()
+        .buildTask(this, task)
+        .schedule();
+  }
+
   @Override
   public final void shutdown() {
     this.metrics.shutdown();

src/main/java/xyz/webmc/originblacklist/velocity/command/OriginBlacklistCommandVelocity.java

@@ -7,18 +7,21 @@ import java.util.List;
 
 import com.velocitypowered.api.command.SimpleCommand;
 
-public class OriginBlacklistCommandVelocity extends OriginBlacklistCommand implements SimpleCommand {
+public final class OriginBlacklistCommandVelocity extends OriginBlacklistCommand implements SimpleCommand {
+  private final OriginBlacklist plugin;
+
   public OriginBlacklistCommandVelocity(OriginBlacklist plugin) {
     super(plugin);
+    this.plugin = plugin;
   }
 
   @Override
-  public void execute(final Invocation invocation) {
-    super.execute(new VCommandContext(invocation));
+  public final void execute(final Invocation invocation) {
+    super.execute(new VCommandContext(this.plugin, invocation));
   }
 
   @Override
-  public List<String> suggest(final Invocation invocation) {
-    return super.suggest(new VCommandContext(invocation));
+  public final List<String> suggest(final Invocation invocation) {
+    return super.suggest(new VCommandContext(this.plugin, invocation));
   }
 }

src/main/java/xyz/webmc/originblacklist/velocity/command/VCommandContext.java

@@ -1,34 +1,42 @@
 package xyz.webmc.originblacklist.velocity.command;
 
+import xyz.webmc.originblacklist.base.OriginBlacklist;
 import xyz.webmc.originblacklist.base.command.CommandContext;
 
 import com.velocitypowered.api.command.SimpleCommand.Invocation;
 import net.kyori.adventure.text.minimessage.MiniMessage;
 
-public class VCommandContext implements CommandContext {
+public final class VCommandContext implements CommandContext {
+  private final OriginBlacklist plugin;
   private final Invocation invocation;
 
-  public VCommandContext(final Invocation invocation) {
+  public VCommandContext(final OriginBlacklist plugin, final Invocation invocation) {
+    this.plugin = plugin;
     this.invocation = invocation;
   }
 
   @Override
-  public String getPlayerName() {
+  public final OriginBlacklist getPlugin() {
+    return this.plugin;
+  }
+
+  @Override
+  public final String getPlayerName() {
     return this.invocation.source().toString();
   }
 
   @Override
-  public void reply(final String message) {
+  public final void reply(final String message) {
     this.invocation.source().sendMessage(MiniMessage.miniMessage().deserialize(message));
   }
 
   @Override
-  public boolean hasPermission(final String permission) {
+  public final boolean hasPermission(final String permission) {
     return this.invocation.source().hasPermission(permission);
   }
 
   @Override
-  public String[] getArgs() {
+  public final String[] getArgs() {
     return this.invocation.arguments();
   }
 }