added gc command, now ignoring miror version, the BufferOverflowException is now fixed

Author: ZetaMap

mod.hjson

@@ -10,7 +10,7 @@ CLaJ (Copy Link and Join) allow you to play with your friends just by creating a
 
 Please check this link for Mindustry v8 compatibility: https://github.com/xpdustry/claj#mindustry-v8-note
 '''
-version: "2.3"
+version: "2.3.1"
 minGameVersion: "146"
 java: true
 hidden: true

server/src/com/xpdustry/claj/server/ClajControl.java

@@ -62,6 +62,13 @@ void registerCommands(ClajRelay server) {
                  "&fr - &lw" + c.description));
     });
 
+    register("gc", "Trigger a garbage collection.", arg -> {
+      int pre = (int)((Runtime.getRuntime().totalMemory() - Runtime.getRuntime().freeMemory()) / 1024 / 1024);
+      System.gc();
+      int post = (int)((Runtime.getRuntime().totalMemory() - Runtime.getRuntime().freeMemory()) / 1024 / 1024);
+      Log.info("@ MB collected. Memory usage now at @ MB.", pre - post, post);
+    });
+    
     register("exit", "Stop the server.", args -> {
       Log.info("Shutting down CLaJ server.");
       server.stop();

server/src/com/xpdustry/claj/server/ClajRelay.java

@@ -118,7 +118,7 @@ public void received(Connection connection, Object object) {
       connection.sendTCP("[scarlet][[CLaJ Server]:[] Your CLaJ version is obsolete! Please update it by "
                        + "installing the 'claj' mod, in the mod browser.");
       connection.close(DcReason.error);
-      Log.warn("Rejected connection @ for incompatible version.", Strings.conIDToString(connection));
+      Log.warn("Rejected room creation of connection @ for incompatible version.", Strings.conIDToString(connection));
 
     } else if (object instanceof ClajPackets.RoomJoinPacket) {
       // Disconnect from a potential another room.
@@ -138,12 +138,14 @@ public void received(Connection connection, Object object) {
     } else if (object instanceof ClajPackets.RoomCreateRequestPacket) {
       // Check the version of client
       String version = ((ClajPackets.RoomCreateRequestPacket)object).version;
-      if (version == null || Strings.isVersionAtLeast(version, Main.serverVersion)) {
+      // Ignore the last part of version, the minor part. (versioning format: 2.major.minor)
+      // The minor part is used when no changes have been made to the client version.
+      if (version == null || Strings.isVersionAtLeast(version, Main.serverVersion, 2)) {
         ClajPackets.ClajMessagePacket p = new ClajPackets.ClajMessagePacket();
         p.message = "Your CLaJ version is outdated, please update it by reinstalling the 'claj' mod.";
         connection.sendTCP(p);
         connection.close(DcReason.error);
-        Log.warn("Rejected connection @ for outdated version.", Strings.conIDToString(connection));
+        Log.warn("Rejected room creation of connection @ for outdated version.", Strings.conIDToString(connection));
         return;
       }
 

server/src/com/xpdustry/claj/server/ClajRoom.java

@@ -141,10 +141,7 @@ public void close() {
     closed = true; // close before kicking connections, to avoid receiving events
 
     host.close(DcReason.closed);
-    clients.values().forEach(c -> {
-      c.setIdleThreshold(-1f);//try to avoid getting idle events
-      c.close(DcReason.closed);
-    });
+    clients.values().forEach(c -> c.close(DcReason.closed));
     clients.clear();
   }
 

server/src/com/xpdustry/claj/server/util/Strings.java

@@ -12,19 +12,29 @@
 
 
 public class Strings extends arc.util.Strings {
+  
+  public static boolean isVersionAtLeast(String currentVersion, String newVersion) {
+    return isVersionAtLeast(currentVersion, newVersion, 0);
+  }
   /** 
-   * @return whether {@code newVersion} is greater than {@code currentVersion}, e.g. "v146" > "124.1"
-   * @apiNote can handle multiple dots in the version, and it's very fast because it only does one iteration.
+   * Compare if the {@code newVersion} is greater than the {@code currentVersion}, e.g. "v146" > "124.1". <br>
+   * {@code maxDepth} limits the number of comparisons of version segments, allowing sub-versions to be ignored. 
+   * (default is 0)
+   * 
+   * @apiNote can handle multiple dots in the version and the 'v' prefix, 
+   *          and it's very fast because it only does one iteration.
    */
-  public static boolean isVersionAtLeast(String currentVersion, String newVersion) {
+  public static boolean isVersionAtLeast(String currentVersion, String newVersion, int maxDepth) {
+    if (maxDepth < 1) maxDepth = Integer.MAX_VALUE;
+    
     int last1 = currentVersion.startsWith("v") ? 1 : 0, 
         last2 = newVersion.startsWith("v") ? 1 : 0, 
         len1 = currentVersion.length(), 
         len2 = newVersion.length(),
         dot1 = 0, dot2 = 0, 
         p1 = 0, p2 = 0;
 
-    while ((dot1 != -1  && dot2 != -1) && (last1 < len1 && last2 < len2)) {
+    while ((dot1 != -1 && dot2 != -1) && (last1 < len1 && last2 < len2) && maxDepth-- > 0) {
       dot1 = currentVersion.indexOf('.', last1);
       dot2 = newVersion.indexOf('.', last2);
       if (dot1 == -1) dot1 = len1;
@@ -37,6 +47,7 @@ public static boolean isVersionAtLeast(String currentVersion, String newVersion)
 
       if (p1 != p2) return p2 > p1;
     }
+    if (maxDepth <= 0) return p2 > p1;
 
     // Continue iteration on newVersion to see if it's just leading zeros.
     while (dot2 != -1 && last2 < len2) {