Fixed Listening to Handshake Packets & ServerPingRecord

[LOOHP]

On version 1.20.6, ChannelProtocolUtil is unable to get the Handshake Protocol PacketType due to Minecraft using a different handler during this phase.

ServerPingRecord is also unable to locate the server icon class on Mojang-mapped servers.

Fixes #2647

[Ingrim4]

Why change the type here from Object to PacketType.Protocol? Doesn't this only introduce duplicate code since you copied the PacketType.Protocol::fromVanilla method to the resolver implementations?

[LOOHP]

ProtocolLib/src/main/java/com/comphenix/protocol/injector/netty/channel/ChannelProtocolUtil.java

Lines 150 to 156 in 9ebfe0d

	public PacketType.Protocol apply(Channel channel, PacketType.Sender sender) {
	String key = this.getKeyForSender(sender);
	Object codecHandler = channel.pipeline().get(key);
	if (codecHandler == null) {
	String unconfiguratedKey = this.getUnconfiguratedKeyForSender(sender);
	if (channel.pipeline().get(unconfiguratedKey) != null) {
	return PacketType.Protocol.HANDSHAKING;

It does create some duplication but as PacketType.Protocol::fromVanilla isn't used everywhere anymore after this change with some cases returning the the packet type directly, I felt like it made sense to make the function return the type directly.