ocelotpotpie / FreedomChat

Liberate your server from the chat-reporting bourgeoisie! Disable chat signing server-side
https://modrinth.com/plugin/freedomchat
GNU General Public License v3.0
204 stars 24 forks source link

Doesn't work. #29

Closed MagmaBro123 closed 1 year ago

MagmaBro123 commented 1 year ago

Title. NCR says chat is still signed, meaning the plugin isn't working, see https://github.com/Aizistral-Studios/No-Chat-Reports/issues/402.

e-im commented 1 year ago

That isn't saying the received chat is still signed, it's saying that the server rejected an unsigned message sent by the client. https://github.com/Aizistral-Studios/No-Chat-Reports/blob/1.19.4-Unified/src/main/java/com/aizistral/nochatreports/common/mixins/client/MixinChatListener.java#L43-L71

FreedomChat doesn't do anything to disable the server checking for signed messages--because the ability to do this is built in to the vanilla server. If you would like the server to accept unsigned messages, disable enforce-secure-profiles in server.properties.

Aizistral commented 1 year ago

I'd argue that it would be a more user-friendly design to automatically turn enforce-secure-profiles off if user goes as far to install a plugin that, as they may think, should rid their server of any chat reporting stuff, of which enforcing message signing is one. I understand why you wouldn't want to go through the trouble of hacking vanilla to do that, but in that case I'd advise to clearly communicate that this is something the user still has to do separately, in readme/project page/etc.

MagmaBro123 commented 1 year ago

I thought this worked without needing to do that until now.

e-im commented 1 year ago

You're right. That isn't communicated very well. I've added a note about this to the top of the README and the Modrinth page.

For anyone reading this in the future: doing this automatically (and especially editing the file like some others do) isn't something I'd accept a PR for.