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.

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

Better handling of multi-line MotD #7

Open
magicus opened this issue Jun 10, 2024 · 1 comment
Open

Better handling of multi-line MotD #7

magicus opened this issue Jun 10, 2024 · 1 comment

Comments

@magicus
Copy link

magicus commented Jun 10, 2024

Our family server has a two-line MotD, used by inserting \n in the motd property. (This is rather common practice for Minecraft servers).

Unfortunately this is shown in the LAN broadcast as the LF unicode block "[LF]". I think a better approach would be to replace this with -, as it will work as an acceptable one-line separator.

This should be a trivial addition to sanitizeMOTD. I can submit a PR, if you like. Let me know.

@Gunni
Copy link
Contributor

Gunni commented Jun 11, 2024

Hey thanks for the issue!

Yes that sounds like a trivial change, feel free to make a PR!

I am rather busy but I'll try to get around to releasing it once you do!

If you don't, I might get around to it some day 😁

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

No branches or pull requests

2 participants