Github user jbertram commented on the issue:
https://github.com/apache/activemq-artemis/pull/1771
Aside from the failing tests this looks OK.
That said, I would love to see us move all our password masking to use the
"ENC(<hash>)" syntax instead of using boolean "mask-password" attributes
everywhere if possible. This is done for the properties login module and it's
very clean. If the text follows the pattern then it should be treated as
masked otherwise it shouldn't.---
