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/03/01 19:51:59 UTC

[GitHub] [activemq] cshannon edited a comment on pull request #714: AMQ-8317 add config param to toggle inclusive terms in logs

cshannon edited a comment on pull request #714:
URL: https://github.com/apache/activemq/pull/714#issuecomment-1055749169


   Agree with @jbonofre, I think having a configuration option makes no sense. ~~Just change it but at this point it should be target for 5.18 as 5.17 should be frozen except for critical stuff~~ I think we can include it in 5.17.0 since it's just logging if you want to but in my opinion it should have the toggle/flag option removed as discussed on the dev list thread.


-- 
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