Skip to content

Broken Chat V2 #3274

Closed
Closed

Description

@BugabuseMe

Are you using MySQL?

Yes

Are you using a proxy?

Yes - Velocity

"/version ChatControlRed" - plugin version

11.4.6

Optional: Error log

https://pastebin.com/0JSUqzBr

ZIP of "/chc debug"

Sorry, didn't include the full thing because I didn't even know where it started, done that on a fresh log, it starts from here above in the issue.
So, if there is chatcontrol, it triggers. I'm thinking it might be because the plugin tried to translate the &a in the link to <green or something and maybe it gets read as something else.

If there is no protocollib, the error triggers anyways, it just doesn't send the spam here

Image Image Image

and instead it doesn't send a message at all.

If there is no chatcontrol, it works

Image

If colors are disabled in chatcontrol, it works

Image

I don't think it's chatcontrol itself as a plugin, I think it's chc trying to translate that &a, because if I remove it from the link I'm sending nothing happens.

This cannot be fixed with the rules tho, from what I've tried.

Even as a non admin, I can send the thing without issues, but STILL the plugin will try to translate the &a failing and breaking the link, it shouldn't if I don't have the permission to send chat colors try to send the &a, but it removes it from the message entirely as a player.

Positive it's not a paper issue directly because we tried without ChatControl and everything works if nothing gets replaced from the plugin.

Steps to reproduce

Write any youtube link with a & and a working color as an admin to send spam in chat if you have protocollib, or have the message not being sent and the logs full of errors without protocollib.

Activity

  1. devpedrobarbosa commented on Aug 10, 2025

    @devpedrobarbosa
    Contributor

    Hey @BugabuseMe

    Thanks for detailing the issue and testing multiple scenarios.

    Just for clarification, which server software and version are you running?

    This seems to be happening on newer versions only. Just wanted to check.

  2. BugabuseMe commented on Aug 10, 2025

    @BugabuseMe
    Author

    I was running 1.21.6, now with 1.21.4 it's not happening anymore, still if you plan to update keep that bug in mind. Should there be the option also in the future, if you'd be willing to change from protocollib to packetevents for the X, vanished players and other stuff, it does not make big servers lag as nearly as protocollib does. Not hating on plib but the dev let himself go nowadays with the coding, or maybe he's just tired, he updated that thing for decades now

  3. devpedrobarbosa commented on Aug 10, 2025

    @devpedrobarbosa
    Contributor

    @BugabuseMe

    We do have plans to switch from ProtocolLib to PacketEvents.

    That may take some time, though.

    About the bug you mentioned, it will be properly fixed as soon as possible :)

  4. kangarko commented on Aug 12, 2025

    @kangarko
    Owner

    @devpedrobarbosa can you try using regex to colorize each word separatedly if it is not a url and dont translate colors in urls

  5. devpedrobarbosa commented on Aug 15, 2025

    @devpedrobarbosa
    Contributor

    @BugabuseMe
    @kangarko

    This is fixed. Please wait for the next release and let me know if it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions