From 30cca70e56c93833b1cd6c5bbe4acb84369be63e Mon Sep 17 00:00:00 2001
From: rtm516 <rtm516@users.noreply.github.com>
Date: Sat, 7 Oct 2023 21:38:44 +0100
Subject: [PATCH] Tell the server we loaded the resource pack (#4183)

* Tell the server we loaded the resource pack
* Decline resource packs if they are not required

---------

Co-authored-by: onebeastchris <github@onechris.mozmail.com>
---
 .../protocol/java/JavaClientboundResourcePacksPacket.java  | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/org/geysermc/geyser/translator/protocol/java/JavaClientboundResourcePacksPacket.java b/core/src/main/java/org/geysermc/geyser/translator/protocol/java/JavaClientboundResourcePacksPacket.java
index 017502280..cc2733076 100644
--- a/core/src/main/java/org/geysermc/geyser/translator/protocol/java/JavaClientboundResourcePacksPacket.java
+++ b/core/src/main/java/org/geysermc/geyser/translator/protocol/java/JavaClientboundResourcePacksPacket.java
@@ -38,6 +38,11 @@ public class JavaClientboundResourcePacksPacket extends PacketTranslator<Clientb
     @Override
     public void translate(GeyserSession session, ClientboundResourcePackPacket packet) {
         // We need to "answer" this to avoid timeout issues related to resource packs
-        session.sendDownstreamPacket(new ServerboundResourcePackPacket(ResourcePackStatus.DECLINED));
+        // If packs are required, we need to lie to the server that we accepted them, as we get kicked otherwise.
+        if (packet.isRequired()) {
+            session.sendDownstreamPacket(new ServerboundResourcePackPacket(ResourcePackStatus.SUCCESSFULLY_LOADED));
+        } else {
+            session.sendDownstreamPacket(new ServerboundResourcePackPacket(ResourcePackStatus.DECLINED));
+        }
     }
 }