Use defaults when public_key_attr/private_key_attr are unset

backend/ldap.go

@@ -43,11 +43,16 @@ func (c *LDAPQueryConfig) Valid() error {
 		}
 		c.scope = s
 	}
+
+	// Since two different daemons use this code, with separate
+	// configs, and one of them does not need one of these
+	// attributes, it's more practical to set defaults rather than
+	// complain if values are unset.
 	if c.PublicKeyAttr == "" {
-		return errors.New("empty public_key_attr")
+		c.PublicKeyAttr = "userPublicKey"
 	}
 	if c.PrivateKeyAttr == "" {
-		return errors.New("empty public_key_attr")
+		c.PrivateKeyAttr = "userEncryptedKey"
 	}
 	return nil
 }