Skip to content

Add system property to allow longer chat messages on legacy versions#1468

Open
Beaness wants to merge 2 commits intoretrooper:2.0from
Beaness:legacychatlength
Open

Add system property to allow longer chat messages on legacy versions#1468
Beaness wants to merge 2 commits intoretrooper:2.0from
Beaness:legacychatlength

Conversation

@Beaness
Copy link
Copy Markdown
Contributor

@Beaness Beaness commented Feb 25, 2026

Similar to
https://github.com/PaperMC/Velocity/blob/6aff78728c267bbb67b07298b8cf4580fb79fc6f/proxy/src/main/java/com/velocitypowered/proxy/protocol/packet/chat/legacy/LegacyChatPacket.java#L49

LunarClient Apollo API allows you to override the max chat length message for older mc versions. Without this change plugins that listen for chat packets break.

@NoJokeFNA
Copy link
Copy Markdown
Contributor

NoJokeFNA commented Feb 25, 2026

Why don't we keep the current design?
https://github.com/retrooper/packetevents/blob/2.0/api/src/main/java/com/github/retrooper/packetevents/protocol/nbt/NBTLimiter.java#L28

And I'm not sure if this will even get merged just because the "LunarClient Apollo API" allows you to change some values.

@Beaness
Copy link
Copy Markdown
Contributor Author

Beaness commented Feb 25, 2026

Why don't we keep the current design? https://github.com/retrooper/packetevents/blob/2.0/api/src/main/java/com/github/retrooper/packetevents/protocol/nbt/NBTLimiter.java#L28

Fixed

And I'm not sure if this will even get merged just because the "LunarClient Apollo API" allows you to change some values.

Seems like a minor addition that isn't super hard to maintain, the default value also doesn't change with this pr.

@retrooper retrooper added the enhancement New feature or request label Feb 27, 2026
@booky10 booky10 self-requested a review April 7, 2026 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants