From affc1d6e08e4e324bdfe2be788dc8cc7535bca5f Mon Sep 17 00:00:00 2001 From: Shane Freeder Date: Thu, 9 Apr 2026 19:03:33 +0100 Subject: [PATCH] Extend debug logging to client play/config handlers --- .../proxy/connection/client/ClientConfigSessionHandler.java | 4 ++++ .../proxy/connection/client/ClientPlaySessionHandler.java | 4 ++++ 2 files changed, 8 insertions(+) diff --git a/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientConfigSessionHandler.java b/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientConfigSessionHandler.java index 299285826..7c6032c73 100644 --- a/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientConfigSessionHandler.java +++ b/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientConfigSessionHandler.java @@ -33,6 +33,7 @@ import com.velocitypowered.proxy.connection.player.resourcepack.ResourcePackResp import com.velocitypowered.proxy.protocol.MinecraftPacket; import com.velocitypowered.proxy.protocol.ProtocolUtils; import com.velocitypowered.proxy.protocol.StateRegistry; +import com.velocitypowered.proxy.protocol.netty.MinecraftDecoder; import com.velocitypowered.proxy.protocol.netty.MinecraftEncoder; import com.velocitypowered.proxy.protocol.packet.ClientSettingsPacket; import com.velocitypowered.proxy.protocol.packet.KeepAlivePacket; @@ -268,6 +269,9 @@ public class ClientConfigSessionHandler implements MinecraftSessionHandler { @Override public void exception(Throwable throwable) { player.disconnect(Component.translatable("velocity.error.player-connection-error", NamedTextColor.RED)); + if (MinecraftDecoder.DEBUG) { + logger.info("Exception while handling plugin message packet for {}", player, throwable); + } } @Override diff --git a/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientPlaySessionHandler.java b/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientPlaySessionHandler.java index a3485983f..c03d31ae9 100644 --- a/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientPlaySessionHandler.java +++ b/proxy/src/main/java/com/velocitypowered/proxy/connection/client/ClientPlaySessionHandler.java @@ -41,6 +41,7 @@ import com.velocitypowered.proxy.connection.forge.legacy.LegacyForgeConstants; import com.velocitypowered.proxy.connection.player.resourcepack.ResourcePackResponseBundle; import com.velocitypowered.proxy.protocol.MinecraftPacket; import com.velocitypowered.proxy.protocol.StateRegistry; +import com.velocitypowered.proxy.protocol.netty.MinecraftDecoder; import com.velocitypowered.proxy.protocol.packet.BossBarPacket; import com.velocitypowered.proxy.protocol.packet.ClientSettingsPacket; import com.velocitypowered.proxy.protocol.packet.JoinGamePacket; @@ -509,6 +510,9 @@ public class ClientPlaySessionHandler implements MinecraftSessionHandler { public void exception(Throwable throwable) { player.disconnect( Component.translatable("velocity.error.player-connection-error", NamedTextColor.RED)); + if (MinecraftDecoder.DEBUG) { + logger.info("Exception while handling plugin message packet for {}", player, throwable); + } } @Override