streamline multi condition check in #bind_dn_and_bind_pwd_valid?

ManageIQ · Oct 30, 2019 · 4efc07e · 4efc07e
1 parent deebc84
commit 4efc07e
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/tools/miq_config_sssd_ldap/cli_config.rb b/tools/miq_config_sssd_ldap/cli_config.rb
@@ -141,8 +141,7 @@ def ldaphost_and_ldapport_valid?
 
     def bind_dn_and_bind_pwd_valid?
       if opts[:mode] == "ldap" || opts[:ldap_role] == true
-        return false if opts[:bind_dn].nil?
-        return false if opts[:bind_pwd].nil?
+        return false if opts[:bind_dn].nil? || opts[:bind_pwd].nil?
       end
       true
     end