[ZOOKEEPER-2272] Code clean up in ZooKeeperServer and KerberosName

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Won't Fix
• Affects Version/s: 3.5.0
• Fix Version/s: 3.7.0
• Component/s: server
• Assignee: Mohammad Arshad
• Reporter: Mohammad Arshad
• Created: 10/Sep/15 06:36
• Updated: 18/May/22 13:40
• Resolved: 07/Jan/21 11:12

Description

1. Following code in org.apache.zookeeper.server.ZooKeeperServer should be cleaned up. Some how it got missed in code review

   if ((System.getProperty("zookeeper.allowSaslFailedClients") != null)
     &&  (System.getProperty("zookeeper.allowSaslFailedClients").equals("true"))) {
   

   should be replaced with

   if(Boolean.getBoolean("zookeeper.allowSaslFailedClients"))
   

2. Similar code clean up can be done in org.apache.zookeeper.server.auth.KerberosName