On 23.7.2014 14:40, Tomas Babej wrote:
Hi,

when poking in the depths of the baseldap, I found this seemingly
redundant search.

ACK. For the record, before commit f1f1b4e the result was used for wait_for_memberof.

BTW, I think this bit:

        # Ensure our target exists
        result = self.api.Command[self.show_command](keys[-1])['result']
        dn = result['dn']

could be optimized to use get_dn_if_exists.

--
Jan Cholasta

_______________________________________________
Freeipa-devel mailing list
Freeipa-devel@redhat.com
https://www.redhat.com/mailman/listinfo/freeipa-devel

Reply via email to