Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Bug]: Integrated Dynamics Storage Terminals always cause a PacketCodecException on the client #1798

Open
flann opened this issue Dec 25, 2023 · 1 comment
Open

[Bug]: Integrated Dynamics Storage Terminals always cause a PacketCodecException on the client #1798

flann opened this issue Dec 25, 2023 · 1 comment
Labels
Bug Something isn't working In Dev Fixed for next pack release Mod issue Caused by a bug in the mod, not the pack Reported to Dev Bug/issue reported to mod developer TODO ToDo, kinda self explanatory

Comments

@flann
Copy link

flann commented Dec 25, 2023 •
edited
Loading

Possible Fixes

Yes

Modpack Version

1.1.0

What happened?

This is running ATM 8 on a server and connecting with a client. Attempting to use an integrated dynamics storage terminal disconnects the client and shows a "PacketCodecException". Nothing shows up in the server logs other than showing the user disconnecting.

  1. Connect a storage terminal to an inventory.
  2. Right-click the terminal
  3. Client disconnects from the server with a "PacketCodecException". Log trace shows the dreaded "io.netty.handler.codec.EncoderException: java.io.EOFException: fieldSize is too long! Length is 28009, but maximum is 45"

latest.log

https://gist.github.com/flann/00549bf494d38454a8ede9e7dd685747#file-latest-log
@flann flann added the Bug Something isn't working label Dec 25, 2023
@flann
Copy link
Author

flann commented Dec 27, 2023 •
edited
Loading

See CyclopsMC/IntegratedTerminals#112

Looks like there's a work-around. Reducing the terminalStoragePacketMaxInstances from 1024 to a lower value (512, 256, or 128 may work) in /config/integratedterminals-common.toml

I tried 256, and it worked for me.

@Mitchell5200 Mitchell5200 added Mod issue Caused by a bug in the mod, not the pack Reported to Dev Bug/issue reported to mod developer TODO ToDo, kinda self explanatory labels Dec 27, 2023
Mitchell5200 added a commit that referenced this issue Jan 7, 2024
@Mitchell5200
Change terminalStoragePacketMaxInstances, Close #1798
a76b48b
@Mitchell5200 Mitchell5200 added the In Dev Fixed for next pack release label Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Something isn't working In Dev Fixed for next pack release Mod issue Caused by a bug in the mod, not the pack Reported to Dev Bug/issue reported to mod developer TODO ToDo, kinda self explanatory
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@flann @Mitchell5200