Skip to content

Use real vanilla limits for LegacyChat#1502

Merged
electronicboy merged 1 commit intoPaperMC:dev/3.0.0from
Outfluencer:patch-2
Jan 31, 2025
Merged

Use real vanilla limits for LegacyChat#1502
electronicboy merged 1 commit intoPaperMC:dev/3.0.0from
Outfluencer:patch-2

Conversation

@Outfluencer
Copy link
Contributor

@Outfluencer Outfluencer commented Jan 31, 2025

Wouldn't 371e686 break the chat with this limit??

If the proxy reads a message from the backend that is longer than 256 it would disconnect.

Client -> server
< 1.11 has 100
| >= 1.11 has 256

Server -> client has always 262144

Wouldn't 371e686
break the chat with this limit??

If the proxy reads a message from the backend that is longer than 256 it would disconnect.

Client -> server
< 1.11 has 100
>= 1.11 has 256

Server -> client has always 262144
@Outfluencer
Copy link
Contributor Author

@electronicboy right now i think all players will be kicked if the backend sends a message longer than 256 chars

@electronicboy electronicboy merged commit 91bdceb into PaperMC:dev/3.0.0 Jan 31, 2025
1 check passed
@Outfluencer
Copy link
Contributor Author

Outfluencer commented Feb 1, 2025

pls unban me on paper dc i fixed some code 👉👈

@mbax
Copy link

mbax commented Feb 1, 2025

The method for appealing bans is documented in our community guidelines.

WiIIiam278 pushed a commit to WiIIiam278/Velocity that referenced this pull request Mar 24, 2025
Client -> server
< 1.11 has 100
>= 1.11 has 256

Server -> client has always 262144
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants