ryan-highley commented on PR #4135:
URL: 
https://github.com/apache/activemq-artemis/pull/4135#issuecomment-1353551365

   > @ryan-highley, I'm not really in favor of a "useMaskedPassword" property. 
We've deprecated this style of configuration in recent years and consolidated 
on the `ENC()` syntax which can be automatically detected.
   
   Agree wholeheartedly!
   
   The parameter-based approach is documented as being deprecated, and I'm 
perfectly happy to remove its support with the system properties here. I 
included the system property solely for "principle of least surprise" since 
elsewhere the "useMaskedPassword" configurations do still work for backwards 
compatibility.
   
   I'll remove the system property and gladly clean up the associated 
extraneous unit tests!
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to