catch NPE when a disconnection occurs early

CHANGELOG.md

@@ -1,7 +1,6 @@
-## GeyserPackSync changelog: Version 2.0-SNAPSHOT
-
-This is a big one!
+Bugfix for 2.0: Catch NullPointerException if a disconnection happens unexpectedly
 
+2.0 Changelog:
 This release aims to fix all the weird issues where Bedrock players sometimes weren't recognized properly.
 Additionally, this release has tons of internal changes to ensure consistent behavior across the bungee and velocity version.
 

common/src/main/java/net/onebeastchris/geyserpacksync/common/GeyserPackSync.java

@@ -181,9 +181,14 @@ public void handleGeyserLoadResourcePackEvent(SessionLoadResourcePacksEvent even
     }
 
     public void handleDisconnectEvent(SessionDisconnectEvent event) {
-        getLogger().debug(String.format("User %s (%s) disconnected, removing from player pack tracker",
-                event.connection().name(), event.connection().xuid()));
-        this.playerPackTracker.remove(event.connection().xuid());
+        try {
+            getLogger().debug(String.format("User %s (%s) disconnected, removing from player pack tracker",
+                    event.connection().name(), event.connection().xuid()));
+            this.playerPackTracker.remove(event.connection().xuid());
+        } catch (NullPointerException ignored) {
+            // Can occur when disconnection is due to an error in the connection.
+            // Since these players aren't in the cache anyway - hopefully - we yeet them
+        }
     }
 
     public void reload() {