nodece commented on code in PR #18358:
URL: https://github.com/apache/pulsar/pull/18358#discussion_r1018048922
##########
pulsar-client/src/main/java/org/apache/pulsar/client/impl/conf/ClientConfigurationData.java:
##########
@@ -377,6 +380,29 @@ public Authentication getAuthentication() {
public void setAuthentication(Authentication authentication) {
this.authentication = authentication;
}
+
+ /**
+ * Set the authentication using auth-related configuration properties if
they are available.
+ * <p>
+ * The {@code authPluginClassName} and one of {@code authParams} or {@code
authParamMap} must be configured for
+ * the authentication to be determined and set.
+ *
+ * @see AuthenticationFactory#create
+ * @throws PulsarClientException if authentication creation fails
+ */
+ public void setAuthenticationFromPropsIfAvailable() throws
PulsarClientException {
Review Comment:
It looks like we move this to `getAuthentication()`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]