ZOOKEEPER-3590: Zookeeper is unable to set the zookeeper.sasl.client.canonicalize.hostname using system property

aristotelhs · anmolnar · commit 0b9d30faedd2 · 2019-11-22T23:42:31.000+01:00
See https://issues.apache.org/jira/browse/ZOOKEEPER-3590 This is a very small patch that gives sets the option after reading it from the system.getProperty(). There should not be any side effects since this is mostly copy paste. Author: aristotelhs <aristotelhs@lenses.io> Reviewers: eolivelli@apache.org, andor@apache.org Closes #1124 from aristotelhs/ZOOKEEPER-3590
diff --git a/zookeeper-server/src/main/java/org/apache/zookeeper/client/ZKClientConfig.java b/zookeeper-server/src/main/java/org/apache/zookeeper/client/ZKClientConfig.java
@@ -99,6 +99,7 @@ protected void handleBackwardCompatibility() {
          * backward compatibility for client specific properties
          */
         setProperty(ZK_SASL_CLIENT_USERNAME, System.getProperty(ZK_SASL_CLIENT_USERNAME));
+        setProperty(ZK_SASL_CLIENT_CANONICALIZE_HOSTNAME, System.getProperty(ZK_SASL_CLIENT_CANONICALIZE_HOSTNAME));
         setProperty(LOGIN_CONTEXT_NAME_KEY, System.getProperty(LOGIN_CONTEXT_NAME_KEY));
         setProperty(ENABLE_CLIENT_SASL_KEY, System.getProperty(ENABLE_CLIENT_SASL_KEY));
         setProperty(ZOOKEEPER_SERVER_REALM, System.getProperty(ZOOKEEPER_SERVER_REALM));
