Add a client config for attempt_udp_networking

Author: ryanhcode

common/src/main/java/dev/ryanhcode/sable/SableClientConfig.java

@@ -21,6 +21,7 @@ public final class SableClientConfig {
 
     public static final ModConfigSpec SPEC;
 
+    public static final ModConfigSpec.BooleanValue ATTEMPT_UDP_NETWORKING;
     public static final ModConfigSpec.BooleanValue SUB_LEVEL_DYNAMIC_SHADING;
     public static final ModConfigSpec.BooleanValue SUB_LEVEL_WATER_OCCLUSION;
     public static final ModConfigSpec.BooleanValue SUB_LEVEL_SKYLIGHT_SHADOWS;
@@ -31,6 +32,7 @@ public final class SableClientConfig {
     static {
         final ModConfigSpec.Builder builder = new ModConfigSpec.Builder();
 
+
         SUB_LEVEL_DYNAMIC_SHADING = builder
                 .comment("Whether sub-levels should apply block shading dynamically")
                 .define("sub_level_dynamic_shading", true);
@@ -51,7 +53,9 @@ public final class SableClientConfig {
         ZOOM_SENSITIVITY = builder
                 .comment("The zoom sensitivity for sub-level camera types")
                 .defineInRange("sub_level_zoom_sensitivity", 0.2, 0.0, 100.0);
-
+        ATTEMPT_UDP_NETWORKING = builder
+                .comment("If Sable should attempt to establish a UDP connection with the server, to receive sub-level movement data over a UDP channel")
+                .define("attempt_udp_networking", true);
 
         SPEC = builder.build();
     }

common/src/main/java/dev/ryanhcode/sable/SableConfig.java

@@ -56,7 +56,7 @@ public final class SableConfig {
                 .comment("If the entire Sable UDP Networking pipeline should be disabled. This can improve compatibility with certain mods like Replay mod and certain networking setups, but will have worse performance and latency for networking sub-levels.")
                 .define("disable_udp_pipeline", false);
         ATTEMPT_UDP_NETWORKING = builder
-                .comment("If Sable should attempt to authenticate with clients and send them sub-level data over UDP")
+                .comment("If Sable should attempt to authenticate with clients and send them sub-level movement data over UDP")
                 .define("attempt_udp_networking", true);
         SUB_LEVEL_SAVING_LOG_MESSAGE = builder
                 .comment("If Sable should log when saving sub-levels for a dimension.")

common/src/main/java/dev/ryanhcode/sable/network/packets/tcp/ClientboundSableUDPActivationPacket.java

@@ -1,6 +1,7 @@
 package dev.ryanhcode.sable.network.packets.tcp;
 
 import dev.ryanhcode.sable.Sable;
+import dev.ryanhcode.sable.SableClientConfig;
 import dev.ryanhcode.sable.mixinterface.udp.ConnectionExtension;
 import dev.ryanhcode.sable.network.packets.udp.SableUDPAuthenticationPacket;
 import dev.ryanhcode.sable.network.tcp.SableTCPPacket;
@@ -39,14 +40,19 @@ public Type<? extends CustomPacketPayload> type() {
 
     @Override
     public void handle(final PacketContext context) {
+        if (!SableClientConfig.ATTEMPT_UDP_NETWORKING.get()) {
+            Sable.LOGGER.info("Received UDP authentication request, ignoring due to disabled attempt_udp_networking config");
+            return;
+        }
+
         final Connection connection = Minecraft.getInstance().getConnection().getConnection();
         final ConnectionExtension connectionExtension = (ConnectionExtension) connection;
         final Channel channel = connectionExtension.sable$getUDPChannel();
 
         final InetSocketAddress baseAddress = ((InetSocketAddress) connection.getRemoteAddress());
         final InetSocketAddress remoteAddress = new InetSocketAddress(baseAddress.getAddress(), baseAddress.getPort());
 
-        Sable.LOGGER.info("Received authentication request, sending response over UDP to {}", remoteAddress);
+        Sable.LOGGER.info("Received UDP authentication request, sending response over UDP to {}", remoteAddress);
 
         channel.eventLoop().execute(() -> {
             final SableUDPAuthenticationPacket packet = new SableUDPAuthenticationPacket(this.uuid.toString());