Backport more upstream fixes

Samsuik · Aug 25, 2024 · 445a7ae · 445a7ae
1 parent dc8c0bb
commit 445a7ae
Show file tree

Hide file tree

Showing 2 changed files with 77 additions and 0 deletions.
diff --git a/patches/api/0014-PAPER-Improve-standard-messenger-logging.patch b/patches/api/0014-PAPER-Improve-standard-messenger-logging.patch
@@ -0,0 +1,57 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Nassim Jahnke <[email protected]>
+Date: Mon, 12 Aug 2024 18:42:08 +0200
+Subject: [PATCH] PAPER Improve standard messenger logging
+
+Backported https://github.com/PaperMC/Paper/commit/7c9240f4a63b13be1fdcedbfb0270f9b49b75518
+
+diff --git a/src/main/java/org/bukkit/plugin/messaging/ChannelNameTooLongException.java b/src/main/java/org/bukkit/plugin/messaging/ChannelNameTooLongException.java
+index 80ef8a2a342ff9dc16c98b5254cadd539f6ea0d9..ea73d9cd9d9d84731c083089d0b9913b4b1f92b6 100644
+--- a/src/main/java/org/bukkit/plugin/messaging/ChannelNameTooLongException.java
++++ b/src/main/java/org/bukkit/plugin/messaging/ChannelNameTooLongException.java
+@@ -9,7 +9,9 @@ public class ChannelNameTooLongException extends RuntimeException {
+         super("Attempted to send a Plugin Message to a channel that was too large. The maximum length a channel may be is " + Messenger.MAX_CHANNEL_SIZE + " chars.");
+     }
+
+-    public ChannelNameTooLongException(String channel) {
+-        super("Attempted to send a Plugin Message to a channel that was too large. The maximum length a channel may be is " + Messenger.MAX_CHANNEL_SIZE + " chars (attempted " + channel.length() + " - '" + channel + ".");
++    // Paper start
++    public ChannelNameTooLongException(int length, String shortenedChannel) {
++        super("Attempted to send a Plugin Message to a channel that was too large. The maximum length a channel may be is " + Messenger.MAX_CHANNEL_SIZE + " chars (attempted " + length + " - '" + shortenedChannel + ".");
++        // Paper end
+     }
+ }
+diff --git a/src/main/java/org/bukkit/plugin/messaging/StandardMessenger.java b/src/main/java/org/bukkit/plugin/messaging/StandardMessenger.java
+index 6fda7f3aa68e76af64362e9afed70fc6a5e92986..230ecf9f9752b0369da3ba7aa6ce68b34a0814c5 100644
+--- a/src/main/java/org/bukkit/plugin/messaging/StandardMessenger.java
++++ b/src/main/java/org/bukkit/plugin/messaging/StandardMessenger.java
+@@ -496,19 +496,26 @@ public class StandardMessenger implements Messenger {
+         if (channel.equals("bungeecord:main")) {
+             return "BungeeCord";
+         }
++        // Paper start - improve error message
+         if (channel.length() > Messenger.MAX_CHANNEL_SIZE) {
+-            throw new ChannelNameTooLongException(channel);
++            throw new ChannelNameTooLongException(channel.length(), shortened(channel));
+         }
+         if (channel.indexOf(':') == -1) {
+-            throw new IllegalArgumentException("Channel must contain : separator (attempted to use " + channel + ")");
++            throw new IllegalArgumentException("Channel must contain : separator (attempted to use " + shortened(channel) + ")");
+         }
+         if (!channel.toLowerCase(Locale.ROOT).equals(channel)) {
+             // TODO: use NamespacedKey validation here
+-            throw new IllegalArgumentException("Channel must be entirely lowercase (attempted to use " + channel + ")");
++            throw new IllegalArgumentException("Channel must be entirely lowercase (attempted to use " + shortened(channel) + ")");
+         }
+         return channel;
+     }
+
++    private static String shortened(String channel) {
++        channel = org.apache.commons.lang3.StringUtils.normalizeSpace(channel);
++        return channel.length() > 32 ? channel.substring(0, 32) + "..." : channel;
++    }
++    // Paper end - improve error message
++
+     /**
+      * Validates the input of a Plugin Message, ensuring the arguments are all
+      * valid.
diff --git a/patches/server/0083-PAPER-Improve-standard-messenger-logging.patch b/patches/server/0083-PAPER-Improve-standard-messenger-logging.patch
@@ -0,0 +1,20 @@
+From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
+From: Nassim Jahnke <[email protected]>
+Date: Mon, 12 Aug 2024 18:42:08 +0200
+Subject: [PATCH] PAPER Improve standard messenger logging
+
+Backported https://github.com/PaperMC/Paper/commit/7c9240f4a63b13be1fdcedbfb0270f9b49b75518
+
+diff --git a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+index 80c8d59566e9e6f0a2ea89b075981a13b65883b0..e530c7f6d9a16b049045f5f70b6b79dbe3a90410 100644
+--- a/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
++++ b/src/main/java/org/bukkit/craftbukkit/entity/CraftPlayer.java
+@@ -2507,7 +2507,7 @@ public class CraftPlayer extends CraftHumanEntity implements Player {
+     }
+
+     public void addChannel(String channel) {
+-        Preconditions.checkState(DISABLE_CHANNEL_LIMIT || this.channels.size() < 128, "Cannot register channel '%s'. Too many channels registered!", channel); // Paper - flag to disable channel limit
++        Preconditions.checkState(DISABLE_CHANNEL_LIMIT || this.channels.size() < 128, "Cannot register channel. Too many channels registered!"); // Paper - flag to disable channel limit
+         channel = StandardMessenger.validateAndCorrectChannel(channel);
+         if (this.channels.add(channel)) {
+             this.server.getPluginManager().callEvent(new PlayerRegisterChannelEvent(this, channel));