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

aristotelhs · nkalmar · commit 2ba38e3c9ced · 2021-04-21T15:49:16.000+02: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 apache#1124 from aristotelhs/ZOOKEEPER-3590 Change-Id: Ic45a2194cbed29754d57c5355148903e9838c13b
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));
