aboutsummaryrefslogtreecommitdiffhomepage
path: root/patches/server/0929-Prevent-causing-expired-keys-from-impacting-new-join.patch
diff options
context:
space:
mode:
Diffstat (limited to 'patches/server/0929-Prevent-causing-expired-keys-from-impacting-new-join.patch')
-rw-r--r--patches/server/0929-Prevent-causing-expired-keys-from-impacting-new-join.patch59
1 files changed, 59 insertions, 0 deletions
diff --git a/patches/server/0929-Prevent-causing-expired-keys-from-impacting-new-join.patch b/patches/server/0929-Prevent-causing-expired-keys-from-impacting-new-join.patch
new file mode 100644
index 0000000000..920fe148a5
--- /dev/null
+++ b/patches/server/0929-Prevent-causing-expired-keys-from-impacting-new-join.patch
@@ -0,0 +1,59 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Shane Freeder <[email protected]>
+Date: Mon, 3 Apr 2023 08:55:52 +0100
+Subject: [PATCH] Prevent causing expired keys from impacting new joins
+
+
+diff --git a/src/main/java/net/minecraft/network/protocol/game/ClientboundPlayerInfoUpdatePacket.java b/src/main/java/net/minecraft/network/protocol/game/ClientboundPlayerInfoUpdatePacket.java
+index 40ac5cf19934207f06755fbea02d4626d7aa0511..32e3f347e2e025e2675e9410116386e472bd6a79 100644
+--- a/src/main/java/net/minecraft/network/protocol/game/ClientboundPlayerInfoUpdatePacket.java
++++ b/src/main/java/net/minecraft/network/protocol/game/ClientboundPlayerInfoUpdatePacket.java
+@@ -96,7 +96,13 @@ public class ClientboundPlayerInfoUpdatePacket implements Packet<ClientGamePacke
+ INITIALIZE_CHAT((serialized, buf) -> {
+ serialized.chatSession = buf.readNullable(RemoteChatSession.Data::read);
+ }, (buf, entry) -> {
+- buf.writeNullable(entry.chatSession, RemoteChatSession.Data::write);
++ // Paper start
++ RemoteChatSession.Data chatSession = entry.chatSession;
++ if (chatSession != null && chatSession.profilePublicKey().hasExpired()) {
++ chatSession = null;
++ }
++ buf.writeNullable(chatSession, RemoteChatSession.Data::write);
++ // Paper end
+ }),
+ UPDATE_GAME_MODE((serialized, buf) -> {
+ serialized.gameMode = GameType.byId(buf.readVarInt());
+diff --git a/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java b/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java
+index f06cbdef43e70cde315b0b4cb2d124efa1071618..ba257447048b9322defaecfd7424579f287ff483 100644
+--- a/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java
++++ b/src/main/java/net/minecraft/server/network/ServerGamePacketListenerImpl.java
+@@ -284,6 +284,7 @@ public class ServerGamePacketListenerImpl extends ServerCommonPacketListenerImpl
+ private final AtomicReference<Instant> lastChatTimeStamp;
+ @Nullable
+ private RemoteChatSession chatSession;
++ private boolean hasLoggedExpiry = false; // Paper
+ private SignedMessageChain.Decoder signedMessageDecoder;
+ private final LastSeenMessagesValidator lastSeenMessages;
+ private final MessageSignatureCache messageSignatureCache;
+@@ -393,6 +394,13 @@ public class ServerGamePacketListenerImpl extends ServerCommonPacketListenerImpl
+ this.disconnect(Component.translatable("multiplayer.disconnect.idling"), org.bukkit.event.player.PlayerKickEvent.Cause.IDLING); // Paper - kick event cause
+ }
+
++ // Paper start
++ if (!hasLoggedExpiry && this.chatSession != null && this.chatSession.profilePublicKey().data().hasExpired()) {
++ LOGGER.info("Player profile key for {} has expired!", this.player.getName().getString());
++ hasLoggedExpiry = true;
++ }
++ // Paper end
++
+ }
+
+ public void resetPosition() {
+@@ -3413,6 +3421,7 @@ public class ServerGamePacketListenerImpl extends ServerCommonPacketListenerImpl
+
+ private void resetPlayerChatState(RemoteChatSession session) {
+ this.chatSession = session;
++ this.hasLoggedExpiry = false; // Paper
+ this.signedMessageDecoder = session.createMessageDecoder(this.player.getUUID());
+ this.chatMessageChain.append((executor) -> {
+ this.player.setChatSession(session);