Pavel Zuna wrote:
Rob Crittenden wrote:None of the LDAP* commands base classes allow the user to set their own uuid. If uuid_attribute is set in LDAPObject, LDAPCreate generates the uuid automatically (using `str(uuid.uuid1())`) and LDAPSearch allows searching by uuid.Pavel Zůna wrote:- remove obsolete code related to PluginProxy- remove parent_key attribute, for the purpose of nested objects the parent's primary key is retrieved automatically- added support for auto-generating UUIDs - make use of the improved attribute printing in CLI !!! depends on previous patch !!! - make LDAPDelete delete all sub-entries, not just one-level - minor bug fixesThis patch is pretty important, it makes little functional difference to IPA by itself, but it is going to be required by subsequent patches that make plugins use the baseldap classes.PavelnackI don't think we want the user to be able to set their own uuid. We want this assigned by the system.
Ok, I missed where it was limited to searching. ack, pushed to master rob
Otherwise I think it looks fine. Deleting a subtree still makes me nervous but since it is called within the framework I think it should be ok.robPavel
smime.p7s
Description: S/MIME Cryptographic Signature
_______________________________________________ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
