Skip to content

Temporary bandaid for &

What does this MR do and why?

Currently bleach is bleaching &. This causes issues with pathing and with minecraft color codes. This change will replace & with & after bleach is done with it. Consulted our resident cyber security expert @Ptarrant1 on this. It seems this will be safe for now, but we should try to replace this on a refactor where we replace & on the way out of the DB instead of before it. That is on the roadmap.

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 Andrew

Merge request reports

Loading