Fix problematic MOTDs on bedrock servers breaking stats collection
What does this MR do and why?
Resolves #530 (closed)
Screenshots or screen recordings
See bug report
How to set up and validate locally
- create bedrock server
- set a non-ASCII character in the MOTD
- verify that no errors are seen in the
commander.logor terminal and correct MOTD shows up in GUI
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
Have you checked this doesn't interfere/conflict/duplicate someone elses work? -
Have you fully tested your changes? -
Have you resolved any lint issues? -
Have you assigned a reviewer? -
Have you applied correct labels?
Edited by Iain Powrie