v1.3.6 — presence pulse 다중 hook 으로 false negative 회피
음악퀴즈 데이터팩의 `#server mq_chat_mod` 가드가 일부 호스트에서 false negative 로 시작 차단되던 문제. 기존엔 매 server tick 한 곳에서만 markModPresence 를 호출했는데, banner/mohist 같은 fabric-bukkit 하이브리드 호스트에서 ServerTickEvents.END_SERVER_TICK 이 안 들어와 점수가 영영 1 로 안 올라갔음. 이번 변경: presence pulse 호출 지점을 셋으로 확장 — 어느 한 이벤트만 firing 돼도 가드가 통과. - ServerLifecycleEvents.SERVER_STARTED (fabric) / ServerStartedEvent (neoforge) — 서버 부팅 완료 직후 한 번 - onPlayerJoin — 플레이어 로그인 시점 (server tick 가 죽어도 발화) - ServerTickEvents.END_SERVER_TICK — 정상 호스트에서의 steady-state ChatAnswerCore 에 onServerStarted public method 추가, onPlayerJoin 에서도 player.level().getServer() 로 server 받아 markModPresence 호출. 세 로더 entrypoint (fabric-1216, fabric-2612, neoforge-1216) 모두에서 SERVER_STARTED 등록. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
@@ -37,8 +37,14 @@ public final class ChatAnswerCore {
|
|||||||
* 본 모드는 서버 측에서 채팅을 가로채는 server-only 모드 — 클라이언트는
|
* 본 모드는 서버 측에서 채팅을 가로채는 server-only 모드 — 클라이언트는
|
||||||
* 설치할 필요가 없고 server 한 곳에 있으면 모든 플레이어에게 적용된다.
|
* 설치할 필요가 없고 server 한 곳에 있으면 모든 플레이어에게 적용된다.
|
||||||
* 따라서 per-player 검증은 무의미하고, fake player {@link #PRESENCE_HOLDER}
|
* 따라서 per-player 검증은 무의미하고, fake player {@link #PRESENCE_HOLDER}
|
||||||
* 점수만 매 server tick 마다 1 로 set 한다. 데이터팩의 start 가드는
|
* 점수만 1 로 set 한다. 데이터팩의 start 가드는
|
||||||
* `score <PRESENCE_HOLDER> <OBJECTIVE> matches 1` 로 검사. */
|
* `score <PRESENCE_HOLDER> <OBJECTIVE> matches 1` 로 검사.
|
||||||
|
*
|
||||||
|
* presence pulse 는 여러 이벤트에서 중복 호출한다 — banner/mohist 같은
|
||||||
|
* fabric-bukkit 하이브리드 호스트에서 일부 Fabric 이벤트(특히
|
||||||
|
* ServerTickEvents.END_SERVER_TICK) 가 안 들어오는 케이스가 보고됨.
|
||||||
|
* SERVER_STARTED / PlayerJoin / TickEnd 셋 중 하나라도 firing 되면
|
||||||
|
* 데이터팩 가드가 통과하도록 모든 진입점에서 markModPresence 호출. */
|
||||||
private static final String MOD_PRESENCE_OBJECTIVE = "mq_chat_mod";
|
private static final String MOD_PRESENCE_OBJECTIVE = "mq_chat_mod";
|
||||||
private static final String PRESENCE_HOLDER = "#server";
|
private static final String PRESENCE_HOLDER = "#server";
|
||||||
|
|
||||||
@@ -61,6 +67,17 @@ public final class ChatAnswerCore {
|
|||||||
LOG.info("[{}] onPlayerJoin fired for {}, scheduling notice in {} ticks",
|
LOG.info("[{}] onPlayerJoin fired for {}, scheduling notice in {} ticks",
|
||||||
MOD_ID, name, NOTICE_DELAY_TICKS);
|
MOD_ID, name, NOTICE_DELAY_TICKS);
|
||||||
PENDING_NOTICES.put(player.getUUID(), NOTICE_DELAY_TICKS);
|
PENDING_NOTICES.put(player.getUUID(), NOTICE_DELAY_TICKS);
|
||||||
|
// tick 이벤트가 안 들어오는 호스트 대비 — join 시점에도 presence 한 번 찍는다.
|
||||||
|
MinecraftServer server = player.level().getServer();
|
||||||
|
if (server != null) markModPresence(server);
|
||||||
|
}
|
||||||
|
|
||||||
|
/** 각 로더 entrypoint 가 서버 부팅 완료 시점에 호출. tick 이벤트가
|
||||||
|
* 발화되지 않는 환경(banner/mohist) 에서 최소 한 번은 presence 가 찍히도록.
|
||||||
|
* 데이터팩 load 가 SERVER_STARTED 보다 먼저 끝나므로 objective 도 이미 존재. */
|
||||||
|
public static void onServerStarted(MinecraftServer server) {
|
||||||
|
LOG.info("[{}] onServerStarted fired, marking presence", MOD_ID);
|
||||||
|
markModPresence(server);
|
||||||
}
|
}
|
||||||
|
|
||||||
/** 각 로더 entrypoint 가 매 server tick 마다 호출해야 한다. */
|
/** 각 로더 entrypoint 가 매 server tick 마다 호출해야 한다. */
|
||||||
|
|||||||
@@ -2,6 +2,7 @@ package kr.tkrmagid.chatanswer.fabric;
|
|||||||
|
|
||||||
import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
|
import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
|
||||||
import net.fabricmc.api.ModInitializer;
|
import net.fabricmc.api.ModInitializer;
|
||||||
|
import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents;
|
||||||
import net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents;
|
import net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents;
|
||||||
import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
|
import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
|
||||||
import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
|
import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
|
||||||
@@ -18,11 +19,12 @@ public final class ChatAnswerFabric implements ModInitializer {
|
|||||||
ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
|
ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
|
||||||
ChatAnswerCore.handleChat(sender, message.signedContent())
|
ChatAnswerCore.handleChat(sender, message.signedContent())
|
||||||
);
|
);
|
||||||
|
ServerLifecycleEvents.SERVER_STARTED.register(ChatAnswerCore::onServerStarted);
|
||||||
ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
|
ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
|
||||||
ChatAnswerCore.onPlayerJoin(handler.player)
|
ChatAnswerCore.onPlayerJoin(handler.player)
|
||||||
);
|
);
|
||||||
ServerTickEvents.END_SERVER_TICK.register(ChatAnswerCore::onServerTick);
|
ServerTickEvents.END_SERVER_TICK.register(ChatAnswerCore::onServerTick);
|
||||||
LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + JOIN + TICK", ChatAnswerCore.MOD_ID);
|
LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + SERVER_STARTED + JOIN + TICK", ChatAnswerCore.MOD_ID);
|
||||||
} catch (Throwable t) {
|
} catch (Throwable t) {
|
||||||
LOG.error("[{}] Fabric entrypoint event registration failed", ChatAnswerCore.MOD_ID, t);
|
LOG.error("[{}] Fabric entrypoint event registration failed", ChatAnswerCore.MOD_ID, t);
|
||||||
throw t;
|
throw t;
|
||||||
|
|||||||
@@ -2,6 +2,7 @@ package kr.tkrmagid.chatanswer.fabric;
|
|||||||
|
|
||||||
import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
|
import kr.tkrmagid.chatanswer.core.ChatAnswerCore;
|
||||||
import net.fabricmc.api.ModInitializer;
|
import net.fabricmc.api.ModInitializer;
|
||||||
|
import net.fabricmc.fabric.api.event.lifecycle.v1.ServerLifecycleEvents;
|
||||||
import net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents;
|
import net.fabricmc.fabric.api.event.lifecycle.v1.ServerTickEvents;
|
||||||
import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
|
import net.fabricmc.fabric.api.message.v1.ServerMessageEvents;
|
||||||
import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
|
import net.fabricmc.fabric.api.networking.v1.ServerPlayConnectionEvents;
|
||||||
@@ -18,11 +19,12 @@ public final class ChatAnswerFabric implements ModInitializer {
|
|||||||
ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
|
ServerMessageEvents.ALLOW_CHAT_MESSAGE.register((message, sender, params) ->
|
||||||
ChatAnswerCore.handleChat(sender, message.signedContent())
|
ChatAnswerCore.handleChat(sender, message.signedContent())
|
||||||
);
|
);
|
||||||
|
ServerLifecycleEvents.SERVER_STARTED.register(ChatAnswerCore::onServerStarted);
|
||||||
ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
|
ServerPlayConnectionEvents.JOIN.register((handler, sender, server) ->
|
||||||
ChatAnswerCore.onPlayerJoin(handler.player)
|
ChatAnswerCore.onPlayerJoin(handler.player)
|
||||||
);
|
);
|
||||||
ServerTickEvents.END_SERVER_TICK.register(ChatAnswerCore::onServerTick);
|
ServerTickEvents.END_SERVER_TICK.register(ChatAnswerCore::onServerTick);
|
||||||
LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + JOIN + TICK", ChatAnswerCore.MOD_ID);
|
LOG.info("[{}] Fabric entrypoint registered: ALLOW_CHAT_MESSAGE + SERVER_STARTED + JOIN + TICK", ChatAnswerCore.MOD_ID);
|
||||||
} catch (Throwable t) {
|
} catch (Throwable t) {
|
||||||
LOG.error("[{}] Fabric entrypoint event registration failed", ChatAnswerCore.MOD_ID, t);
|
LOG.error("[{}] Fabric entrypoint event registration failed", ChatAnswerCore.MOD_ID, t);
|
||||||
throw t;
|
throw t;
|
||||||
|
|||||||
@@ -3,7 +3,7 @@ org.gradle.parallel=true
|
|||||||
|
|
||||||
# ───── mod metadata ─────────────────────────────────────────────────────────
|
# ───── mod metadata ─────────────────────────────────────────────────────────
|
||||||
mod_id=chat_answer
|
mod_id=chat_answer
|
||||||
mod_version=1.3.5
|
mod_version=1.3.6
|
||||||
mod_group=kr.tkrmagid.chatanswer
|
mod_group=kr.tkrmagid.chatanswer
|
||||||
mod_name=채팅정답
|
mod_name=채팅정답
|
||||||
|
|
||||||
|
|||||||
@@ -8,12 +8,14 @@ import net.neoforged.fml.common.Mod;
|
|||||||
import net.neoforged.neoforge.common.NeoForge;
|
import net.neoforged.neoforge.common.NeoForge;
|
||||||
import net.neoforged.neoforge.event.ServerChatEvent;
|
import net.neoforged.neoforge.event.ServerChatEvent;
|
||||||
import net.neoforged.neoforge.event.entity.player.PlayerEvent;
|
import net.neoforged.neoforge.event.entity.player.PlayerEvent;
|
||||||
|
import net.neoforged.neoforge.event.server.ServerStartedEvent;
|
||||||
import net.neoforged.neoforge.event.tick.ServerTickEvent;
|
import net.neoforged.neoforge.event.tick.ServerTickEvent;
|
||||||
|
|
||||||
@Mod(ChatAnswerCore.MOD_ID)
|
@Mod(ChatAnswerCore.MOD_ID)
|
||||||
public final class ChatAnswerNeoForge {
|
public final class ChatAnswerNeoForge {
|
||||||
public ChatAnswerNeoForge(IEventBus modBus) {
|
public ChatAnswerNeoForge(IEventBus modBus) {
|
||||||
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerChat);
|
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerChat);
|
||||||
|
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerStarted);
|
||||||
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onPlayerLogin);
|
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onPlayerLogin);
|
||||||
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerTick);
|
NeoForge.EVENT_BUS.addListener(ChatAnswerNeoForge::onServerTick);
|
||||||
}
|
}
|
||||||
@@ -26,6 +28,11 @@ public final class ChatAnswerNeoForge {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@SubscribeEvent
|
||||||
|
public static void onServerStarted(ServerStartedEvent event) {
|
||||||
|
ChatAnswerCore.onServerStarted(event.getServer());
|
||||||
|
}
|
||||||
|
|
||||||
@SubscribeEvent
|
@SubscribeEvent
|
||||||
public static void onPlayerLogin(PlayerEvent.PlayerLoggedInEvent event) {
|
public static void onPlayerLogin(PlayerEvent.PlayerLoggedInEvent event) {
|
||||||
if (event.getEntity() instanceof ServerPlayer player) {
|
if (event.getEntity() instanceof ServerPlayer player) {
|
||||||
|
|||||||
Reference in New Issue
Block a user