From 95d8747bd5f78a6816134030a9d2981ce0a99bf1 Mon Sep 17 00:00:00 2001
From: Dawson <dawson@funkemunky.cc>
Date: Fri, 20 Feb 2026 09:29:52 -0500
Subject: [PATCH] Fixing webhook spam

---
 .../src/main/java/dev/brighten/antivpn/api/APIPlayer.java      | 3 +++
 .../src/main/java/dev/brighten/antivpn/api/VPNExecutor.java    | 2 --
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/Common/Source/src/main/java/dev/brighten/antivpn/api/APIPlayer.java b/Common/Source/src/main/java/dev/brighten/antivpn/api/APIPlayer.java
index 0bf5200..f4d6dce 100644
--- a/Common/Source/src/main/java/dev/brighten/antivpn/api/APIPlayer.java
+++ b/Common/Source/src/main/java/dev/brighten/antivpn/api/APIPlayer.java
@@ -20,6 +20,7 @@ import com.github.benmanes.caffeine.cache.Cache;
 import com.github.benmanes.caffeine.cache.Caffeine;
 import dev.brighten.antivpn.AntiVPN;
 import dev.brighten.antivpn.message.VpnString;
+import dev.brighten.antivpn.webhook.WebhookNotifier;
 import lombok.Getter;
 import lombok.Setter;
 
@@ -92,6 +93,7 @@ public abstract class APIPlayer {
             if(cachedResult.response().getIp().equals(ip.getHostAddress())) {
                 AntiVPN.getInstance().getExecutor().log(Level.FINE, "Cached result for " + ip.getHostAddress() + " is " + cachedResult.resultType());
                 if(cachedResult.resultType().isShouldBlock()) {
+                    WebhookNotifier.sendWebhookNotification(this, cachedResult);
                     AntiVPN.getInstance().getExecutor().handleKickingOfPlayer(cachedResult, this);
                 }
                 onResult.accept(cachedResult);
@@ -135,6 +137,7 @@ public abstract class APIPlayer {
 
                     checkResultCache.put(ip.getHostAddress(), new CheckResult(checkResult.response(), checkResult.resultType(), true));
                     if(checkResult.resultType().isShouldBlock()) {
+                        WebhookNotifier.sendWebhookNotification(this, checkResult);
                         AntiVPN.getInstance().getExecutor().handleKickingOfPlayer(checkResult, this);
                     }
                     onResult.accept(checkResult);
diff --git a/Common/Source/src/main/java/dev/brighten/antivpn/api/VPNExecutor.java b/Common/Source/src/main/java/dev/brighten/antivpn/api/VPNExecutor.java
index 3fb3611..a6f173e 100644
--- a/Common/Source/src/main/java/dev/brighten/antivpn/api/VPNExecutor.java
+++ b/Common/Source/src/main/java/dev/brighten/antivpn/api/VPNExecutor.java
@@ -23,7 +23,6 @@ import dev.brighten.antivpn.utils.Tuple;
 import dev.brighten.antivpn.utils.json.JSONException;
 import dev.brighten.antivpn.web.FunkemunkyAPI;
 import dev.brighten.antivpn.web.objects.VPNResponse;
-import dev.brighten.antivpn.webhook.WebhookNotifier;
 import lombok.Getter;
 
 import java.io.IOException;
@@ -78,7 +77,6 @@ public abstract class VPNExecutor {
 
     public void handleKickingOfPlayer(CheckResult result, APIPlayer player) {
         // Send webhook notification if enabled
-        WebhookNotifier.sendWebhookNotification(player, result);
 
         //Ensuring kick task is always running
         if(kickTask == null || kickTask.isDone() || kickTask.isCancelled()) {