Properly skip over "v6" IP address in compatibility mode

Signed-off-by: Joshua Castle <26531652+Kas-tle@users.noreply.github.com>

Author: Kas-tle

README.md

@@ -13,6 +13,7 @@ You can join the [Discord](https://discord.gg/5z4GuSnqmQ) for help with this for
 - Allows for resetting security state if `Open Connection Reply 1` is resent by the server
 - Only do retries with `Open Connection Request 1`, and reserve `Open Connection Request 2` only as a direct response to `Open Connection Reply 1`
 - Allows using datagram channel factories for raknet (from [@AlexProgrammerDE](https://github.com/AlexProgrammerDE))
+- Skips over improperly typed client address fields
 
 ## Usage
 

gradle.properties

@@ -1,2 +1,2 @@
 # Only update version on publishing to Maven Central
-version=1.2.1
+version=1.2.2

transport-raknet/src/main/java/org/cloudburstmc/netty/handler/codec/raknet/client/RakClientOfflineHandlerCompatible.java

@@ -16,6 +16,8 @@
 import org.cloudburstmc.netty.handler.codec.raknet.common.RakSessionCodec;
 import org.cloudburstmc.netty.util.RakUtils;
 
+import static org.cloudburstmc.netty.channel.raknet.RakConstants.MAXIMUM_MTU_SIZE;
+
 public class RakClientOfflineHandlerCompatible extends RakClientOfflineHandler {
     public static final String NAME = "rak-client-handler";
 

transport-raknet/src/main/java/org/cloudburstmc/netty/util/RakUtils.java

@@ -98,17 +98,15 @@ public static InetSocketAddress readAddress(ByteBuf buffer) {
     }
 
     public static boolean skipAddress(ByteBuf buffer) {
-        short type = buffer.readByte();
+        short type = buffer.readUnsignedByte();
         try {
             if (type == 4) {
                 // Skip 4 + 2 bytes
                 buffer.skipBytes(6);
-            } else if (type == 6) {
+            } else {
                 // Skip 2 + 2 + 4 + 16 + 4 bytes
+                // Vanilla client treats as v6 if unknown type
                 buffer.skipBytes(28);
-            } else {
-                // Vanilla client skips over if the type is not 4 or 6
-                return false;
             }
         } catch (IndexOutOfBoundsException e) {
             return false;