kowshik edited a comment on pull request #9001: URL: https://github.com/apache/kafka/pull/9001#issuecomment-656902905
@dajac Thank you for taking a look! IIUC you are referring to these lines: https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/security/authenticator/SaslServerAuthenticator.java#L550-L553 My requirement is that under the hood of the newly added API: `org.apache.kafka.clients.Admin#describeFeatures`, the `ApiVersionsResponse` returned to the `AdminClient` needs to contain the features information. Note that this new API issues an explicit `ApiVersionsRequest` under the hood. In such a case do you think I should populate the features information in the above lines in `SaslServerAuthenticator` too? I'm trying to understand where would this come into play (sorry I know little to nothing about `SaslServerAuthenticator`). ---------------------------------------------------------------- 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
