rondagostino commented on pull request #10694: URL: https://github.com/apache/kafka/pull/10694#issuecomment-842403412
Thanks, for the review @mumrah! Regarding your question: > Looks like we're checking client security protocol + sasl mechanism as well as the inter broker protocol + sasl mechanism. Do we need to do the same for "intercontroller_security_protocol" in KRaft mode? The answer is yes -- it's a good point. I've opened https://issues.apache.org/jira/browse/KAFKA-12799 to extend the existing tests to apply to KRaft controllers, and I indicated in that ticket that we will have to take security config mutations into account for that implementation as we did here. I hope it's okay that we defer this to when we get to that ticket. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org