Move fix under status handshake

Log when the protocol is changed.
Elytrium · Apr 9, 2024 · 145440c · 145440c
1 parent a22e943
commit 145440c
Showing 1 changed file with 12 additions and 4 deletions.
diff --git a/src/main/java/net/elytrium/fastmotd/injection/HandshakeSessionHandlerHook.java b/src/main/java/net/elytrium/fastmotd/injection/HandshakeSessionHandlerHook.java
@@ -67,11 +67,19 @@ public boolean handle(LegacyHandshakePacket packet) {
 
   @Override
   public boolean handle(HandshakePacket handshake) {
-    if (handshake.getProtocolVersion() == null || handshake.getProtocolVersion() == ProtocolVersion.UNKNOWN) {
-      handshake.setProtocolVersion(ProtocolVersion.MAXIMUM_VERSION);
-    }
-
     if (handshake.getNextStatus() == StateRegistry.STATUS_ID) {
+      if (handshake.getProtocolVersion() == null || handshake.getProtocolVersion() == ProtocolVersion.UNKNOWN) {
+        handshake.setProtocolVersion(ProtocolVersion.MAXIMUM_VERSION);
+
+        if (Settings.IMP.MAIN.LOG_PINGS) {
+          this.plugin.getLogger().info(
+              "Unknown protocol version detected from {}, replaced with version {}",
+              this.connection.getRemoteAddress(),
+              ProtocolVersion.MAXIMUM_VERSION
+          );
+        }
+      }
+
       this.protocolVersion = handshake.getProtocolVersion();
       this.serverAddress = handshake.getServerAddress() + ":" + handshake.getPort();
       this.channel.pipeline().remove(Connections.FRAME_ENCODER);