@rajinisivaram @stanislavkozlovski I like the idea of SaslExtensionsCallback starting out with empty extensions so that if it isn't handled the caller will not receive null; I think this is a reasonable approach for dealing with the backwards compatibility issue. Should I make the change as part of this PR? If so, I think I would add `public static final SaslExtensions NO_SASL_EXTENSIONS = ...` to the SaslExtensions class and refer to that in both the `SaslExtensionsCallback` and `OAuthBearerClientInitialResponse` constructors. (I'll also fix "Update javadoc since there is no return value")
[ Full content available at: https://github.com/apache/kafka/pull/5552 ] This message was relayed via gitbox.apache.org for [email protected]
