ViaVersion / ViaRewind

ViaBackwards addon to allow 1.8.x and 1.7.x clients on newer server versions.
https://hangar.papermc.io/ViaVersion/ViaRewind
GNU General Public License v3.0
249 stars 79 forks source link

1.8.9 disconnect InventoryPacket #329

Closed htbrown closed 3 years ago

htbrown commented 3 years ago

/viaversion dump Output

https://dump.viaversion.com/d374370fa912b69262b91839848cfd6e56c87a457eb55c94bd6bc83ffaf0f7a8

Console Error

[thehtbrown] disconnected with: EncoderException : com.viaversion.viaversion.exception.InformativeException: Please post this error to https://github.com/ViaVersion/ViaVersion/issues and follow the issue template
{Type: Byte, Data: [], Source 0: de.gerrygames.viarewind.protocol.protocol1_8to1_9.packets.InventoryPackets$4 (Anonymous), Packet ID: 47}
Actual Error:  @ io.netty.handler.codec.MessageToMessageEncoder:104

Bug Description

Tested on 1.8.9, installed on a Waterfall server (ViaVersion, ViaBackwards, ViaRewind).

When a player joins on 1.8.9 they are kicked.

Steps to Reproduce

  1. Install latest dev version of ViaVersion, ViaBackwards and ViaRewind on a Waterfall (build 436) server.
  2. Try to connect on a 1.8.9 client (not sure if it happens on any others).

Expected Behavior

Player shouldn't get kicked from the server.

Additional Server Info

Installed on Waterfall with a custom plugin that doesn't affect it.

Checklist

creeper123123321 commented 3 years ago
W ERROR IN Protocol1_8TO1_9 IN REMAP OF SET_SLOT (0x16)

Caused by: com.viaversion.viaversion.exception.InformativeException: Please post this error to https://github.com/ViaVersion/ViaVersion/issues and follow the issue template
{Type: Byte, Data: [], Source 0: de.gerrygames.viarewind.protocol.protocol1_8to1_9.packets.InventoryPackets$4 (Anonymous), Packet ID: 47}
Actual Error: 
Caused by: java.io.IOException: Unable to read type Byte, found Unsigned Byte
        at com.viaversion.viaversion.protocol.packet.PacketWrapperImpl.read(PacketWrapperImpl.java:140) ~[viaaas.jar:?]
        at com.viaversion.viaversion.api.protocol.remapper.PacketRemapper.lambda$map$0(PacketRemapper.java:47) ~[viaaas.jar:?]
        at com.viaversion.viaversion.api.protocol.remapper.PacketRemapper$$Lambda$150/0000000038E29AF0.handle(Unknown Source) ~[?:?]
        at com.viaversion.viaversion.api.protocol.remapper.PacketRemapper.remap(PacketRemapper.java:158) ~[viaaas.jar:?]
        at com.viaversion.viaversion.api.protocol.AbstractProtocol.transform(AbstractProtocol.java:287) ~[viaaas.jar:?]
        at com.viaversion.viaversion.protocol.packet.PacketWrapperImpl.apply(PacketWrapperImpl.java:350) ~[viaaas.jar:?]
        at com.viaversion.viaversion.protocol.packet.PacketWrapperImpl.apply(PacketWrapperImpl.java:338) ~[viaaas.jar:?]
        at com.viaversion.viaversion.protocol.packet.PacketWrapperImpl.apply(PacketWrapperImpl.java:45) ~[viaaas.jar:?]
        at com.viaversion.viaversion.protocol.ProtocolPipelineImpl.transform(ProtocolPipelineImpl.java:117) ~[viaaas.jar:?]
        at com.viaversion.viaversion.connection.UserConnectionImpl.transform(UserConnectionImpl.java:289) ~[viaaas.jar:?]
        at com.viaversion.viaversion.connection.UserConnectionImpl.transformClientbound(UserConnectionImpl.java:268) ~[viaaas.jar:?]
        at com.viaversion.aas.handler.ViaCodec.decode(ViaCodec.kt:20) ~[viaaas.jar:?]
        at com.viaversion.aas.handler.ViaCodec.decode(ViaCodec.kt:10) ~[viaaas.jar:?]
        at io.netty.handler.codec.MessageToMessageCodec$2.decode(MessageToMessageCodec.java:81) ~[viaaas.jar:?]
        at io.netty.handler.codec.MessageToMessageDecoder.channelRead(MessageToMessageDecoder.java:88) ~[viaaas.jar:?]
        ... 36 more
creeper123123321 commented 3 years ago

may be fixed by https://github.com/ViaVersion/ViaRewind/commit/d6e905dfe1eea94fbb08392885965e27fa884252

creeper123123321 commented 3 years ago

fixed in -DEV builds