On 27.2.2013 17:46, Petr Viktorin wrote:
On 02/27/2013 04:00 PM, Jan Cholasta wrote:
I have noticed two things that are not really related to your work, but
here they are nonetheless:
+ if self.admin_conn.get_entries(
+ DN(api.env.container_ranges, self.suffix),
Is that a valid filter?
I don't think it is, fixed.
+ if self.admin_conn.get_entries("cn=accounts," + self.suffix,
+ ldap.SCOPE_SUBTREE, id_filter):
This doesn't seem right as well, why is the DN class not used here?
I made it a DN.
- conn.sasl_interactive_bind_s(None, sasl_auth)
sasl_auth is unused after this change, can you please remove it as well
(and cb_info too)?
- self.sasl_interactive_bind_s, timeout, None, SASL_AUTH)
+ self.conn.sasl_interactive_bind_s, timeout, None, SASL_AUTH)
Again, this is not related to your work, but can we please rename
SASL_AUTH to SASL_GSSAPI?
OK, makes sense
- res = con.search_st(str(base),
+ res = con.get_entries(base, con.SCOPE_SUBTREE, srcfilter, attrs)
I assume the timeout is there for a reason, can you please keep it?
+ entry['originfilter'] = [disable_attr]
I think you forgot to call update_entry here.
Fixed, thanks again.
Freeipa-devel mailing list