Ensure players pass all join checks before adding them to list of Que…

…ry IPs
adryd325 · Aug 24, 2024 · 416c18d · 416c18d
1 parent a0c9aab
commit 416c18d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/build.gradle b/build.gradle
@@ -1,6 +1,6 @@
 plugins {
 	id 'fabric-loom' version '1.2-SNAPSHOT'
-	id 'org.ajoberstar.grgit' version '4.1.0'
+	id 'org.ajoberstar.grgit' version '4.1.1'
 	id 'maven-publish'
 }
 

diff --git a/src/main/java/com/adryd/sneaky/mixin/MixinServerLoginNetworkHandler.java b/src/main/java/com/adryd/sneaky/mixin/MixinServerLoginNetworkHandler.java
@@ -32,7 +32,9 @@ class MixinServerLoginNetworkHandler {
 
     @Inject(method = "tickVerify", at = @At(value = "INVOKE", target = "Lnet/minecraft/server/PlayerManager;checkCanJoin(Ljava/net/SocketAddress;Lcom/mojang/authlib/GameProfile;)Lnet/minecraft/text/Text;", shift = At.Shift.AFTER))
     private void atSuccessfulJoin(CallbackInfo ci) {
-        IPList.INSTANCE.addToIPList(this.connection.getAddress());
+        if (this.server.getPlayerManager().checkCanJoin(this.connection.getAddress(), this.profile) == null) {
+            IPList.INSTANCE.addToIPList(this.connection.getAddress());
+        }
     }
 
     @Inject(method = "onDisconnected", at = @At("HEAD"), cancellable = true)