You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@activemq.apache.org by GitBox <gi...@apache.org> on 2022/02/07 18:27:02 UTC

[GitHub] [activemq-artemis] gemmellr commented on pull request #3851: ARTEMIS-3573 Support PropertiesLoginModule custom password codecs

gemmellr commented on pull request #3851:
URL: https://github.com/apache/activemq-artemis/pull/3851#issuecomment-1031781449


   I left it saying "I think it would be good for others to review and see what they think, e.g on the char[]->String change on the newly exposed verify/compare methods."
   
   (the context of prior discussion would show I would probably just have used the char[] type of the existing method being exposed here)


-- 
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: gitbox-unsubscribe@activemq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org