gemmellr commented on issue #30: PROTON-1998: Add SASL protocol trace URL: https://github.com/apache/qpid-proton-j/pull/30#issuecomment-459771487 Yeah, its a fair point, and I have actually held off doing this in the past precisely because I think so (and also because it couldn't be added fully without disruption to existing usage until we moved to Java 8). It would be more consistent with proton-c to do it, and it is a very specific trace level tool few should use without expecting such detail, which is only showing detail that can be viewed in other ways already..but it may as you say be surprising that it starts happening from proton-j after all these years. Having a separate flag for it seems both reasonable in that regard, and annoying+inconsistent. Rock, hard place :)
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org
