Addition of debug packet configurations.
This commit is contained in:
		| @@ -691,4 +691,24 @@ AltDevShowQuestsLoadInLogs = False | ||||
|  | ||||
| # Show scripts while loading them. | ||||
| # Default: False | ||||
| AltDevShowScriptsLoadInLogs = False | ||||
| AltDevShowScriptsLoadInLogs = False | ||||
|  | ||||
| # Debug incoming packets. | ||||
| # Default: False | ||||
| DebugIncomingPackets = False | ||||
|  | ||||
| # Debug ex-incoming packets. | ||||
| # Default: False | ||||
| DebugExIncomingPackets = False | ||||
|  | ||||
| # Debug outgoing packets. | ||||
| # Default: False | ||||
| DebugOutgoingPackets = False | ||||
|  | ||||
| # Debug unknown packets. | ||||
| # Default: True | ||||
| DebugUnknownPackets = True | ||||
|  | ||||
| # Excluded packet list. | ||||
| # Packet names that are excluded from debugging, separated by commas. | ||||
| ExcludedPacketList = AbnormalStatusUpdate, AcquireSkillList, Attack, AutoAttackStart, AutoAttackStop, DeleteObject, ExAutoSoulShot, ExStorageMaxCount, ExUserBoostStat, ExUserInfoAbnormalVisualEffect, ExUserInfoEquipSlot, MoveToLocation, NpcInfo, NpcSay, SkillCoolTime, SocialAction, StatusUpdate, UserInfo, ValidatePosition | ||||
|   | ||||
| @@ -446,6 +446,11 @@ public class Config | ||||
| 	public static boolean ALT_DEV_NO_SPAWNS; | ||||
| 	public static boolean ALT_DEV_SHOW_QUESTS_LOAD_IN_LOGS; | ||||
| 	public static boolean ALT_DEV_SHOW_SCRIPTS_LOAD_IN_LOGS; | ||||
| 	public static boolean DEBUG_INCOMING_PACKETS; | ||||
| 	public static boolean DEBUG_EX_INCOMING_PACKETS; | ||||
| 	public static boolean DEBUG_OUTGOING_PACKETS; | ||||
| 	public static boolean DEBUG_UNKNOWN_PACKETS; | ||||
| 	public static Set<String> ALT_DEV_EXCLUDED_PACKETS; | ||||
| 	public static int SCHEDULED_THREAD_POOL_COUNT; | ||||
| 	public static int THREADS_PER_SCHEDULED_THREAD_POOL; | ||||
| 	public static int INSTANT_THREAD_POOL_COUNT; | ||||
| @@ -2035,6 +2040,16 @@ public class Config | ||||
| 			ALT_DEV_NO_SPAWNS = generalConfig.getBoolean("AltDevNoSpawns", false) || Boolean.getBoolean("nospawns"); | ||||
| 			ALT_DEV_SHOW_QUESTS_LOAD_IN_LOGS = generalConfig.getBoolean("AltDevShowQuestsLoadInLogs", false); | ||||
| 			ALT_DEV_SHOW_SCRIPTS_LOAD_IN_LOGS = generalConfig.getBoolean("AltDevShowScriptsLoadInLogs", false); | ||||
| 			DEBUG_INCOMING_PACKETS = generalConfig.getBoolean("DebugIncomingPackets", false); | ||||
| 			DEBUG_EX_INCOMING_PACKETS = generalConfig.getBoolean("DebugExIncomingPackets", false); | ||||
| 			DEBUG_OUTGOING_PACKETS = generalConfig.getBoolean("DebugOutgoingPackets", false); | ||||
| 			DEBUG_UNKNOWN_PACKETS = generalConfig.getBoolean("DebugUnknownPackets", true); | ||||
| 			final String[] packets = generalConfig.getString("ExcludedPacketList", "").trim().split(","); | ||||
| 			ALT_DEV_EXCLUDED_PACKETS = new HashSet<>(packets.length); | ||||
| 			for (String packet : packets) | ||||
| 			{ | ||||
| 				ALT_DEV_EXCLUDED_PACKETS.add(packet.trim()); | ||||
| 			} | ||||
| 			ALLOW_DISCARDITEM = generalConfig.getBoolean("AllowDiscardItem", true); | ||||
| 			AUTODESTROY_ITEM_AFTER = generalConfig.getInt("AutoDestroyDroppedItemAfter", 600); | ||||
| 			HERB_AUTO_DESTROY_TIME = generalConfig.getInt("AutoDestroyHerbTime", 60) * 1000; | ||||
|   | ||||
| @@ -21,6 +21,7 @@ import java.util.HashSet; | ||||
| import java.util.Set; | ||||
| import java.util.function.Supplier; | ||||
|  | ||||
| import org.l2jmobius.Config; | ||||
| import org.l2jmobius.commons.network.IConnectionState; | ||||
| import org.l2jmobius.commons.network.IIncomingPacket; | ||||
| import org.l2jmobius.commons.network.IIncomingPackets; | ||||
| @@ -658,7 +659,23 @@ public enum ExIncomingPackets implements IIncomingPackets<GameClient> | ||||
| 	@Override | ||||
| 	public IIncomingPacket<GameClient> newIncomingPacket() | ||||
| 	{ | ||||
| 		return _incomingPacketFactory.get(); | ||||
| 		final IIncomingPacket<GameClient> packet = _incomingPacketFactory.get(); | ||||
| 		if (Config.DEBUG_EX_INCOMING_PACKETS) | ||||
| 		{ | ||||
| 			if (packet != null) | ||||
| 			{ | ||||
| 				final String name = packet.getClass().getSimpleName(); | ||||
| 				if (!Config.ALT_DEV_EXCLUDED_PACKETS.contains(name)) | ||||
| 				{ | ||||
| 					PacketLogger.info("[C EX] " + name); | ||||
| 				} | ||||
| 			} | ||||
| 			else if (Config.DEBUG_UNKNOWN_PACKETS) | ||||
| 			{ | ||||
| 				PacketLogger.info("[C EX] " + _packetId); | ||||
| 			} | ||||
| 		} | ||||
| 		return packet; | ||||
| 	} | ||||
| 	 | ||||
| 	@Override | ||||
|   | ||||
| @@ -234,6 +234,15 @@ public class GameClient extends ChannelInboundHandler<GameClient> | ||||
| 				return; | ||||
| 			} | ||||
| 			 | ||||
| 			if (Config.DEBUG_OUTGOING_PACKETS) | ||||
| 			{ | ||||
| 				final String name = packet.getClass().getSimpleName(); | ||||
| 				if (!Config.ALT_DEV_EXCLUDED_PACKETS.contains(name)) | ||||
| 				{ | ||||
| 					PacketLogger.info("[S] " + name); | ||||
| 				} | ||||
| 			} | ||||
| 			 | ||||
| 			// TODO: Set as parameter to packets used? | ||||
| 			if (Config.MULTILANG_ENABLE) | ||||
| 			{ | ||||
|   | ||||
| @@ -21,6 +21,7 @@ import java.util.HashSet; | ||||
| import java.util.Set; | ||||
| import java.util.function.Supplier; | ||||
|  | ||||
| import org.l2jmobius.Config; | ||||
| import org.l2jmobius.commons.network.IConnectionState; | ||||
| import org.l2jmobius.commons.network.IIncomingPacket; | ||||
| import org.l2jmobius.commons.network.IIncomingPackets; | ||||
| @@ -244,7 +245,23 @@ public enum IncomingPackets implements IIncomingPackets<GameClient> | ||||
| 	@Override | ||||
| 	public IIncomingPacket<GameClient> newIncomingPacket() | ||||
| 	{ | ||||
| 		return _incomingPacketFactory.get(); | ||||
| 		final IIncomingPacket<GameClient> packet = _incomingPacketFactory.get(); | ||||
| 		if (Config.DEBUG_INCOMING_PACKETS) | ||||
| 		{ | ||||
| 			if (packet != null) | ||||
| 			{ | ||||
| 				final String name = packet.getClass().getSimpleName(); | ||||
| 				if (!Config.ALT_DEV_EXCLUDED_PACKETS.contains(name)) | ||||
| 				{ | ||||
| 					PacketLogger.info("[C] " + name); | ||||
| 				} | ||||
| 			} | ||||
| 			else if (Config.DEBUG_UNKNOWN_PACKETS) | ||||
| 			{ | ||||
| 				PacketLogger.info("[C] " + _packetId); | ||||
| 			} | ||||
| 		} | ||||
| 		return packet; | ||||
| 	} | ||||
| 	 | ||||
| 	@Override | ||||
|   | ||||
		Reference in New Issue
	
	Block a user
	 MobiusDevelopment
					MobiusDevelopment