On 2.9.2014 17:22, Nathaniel McCallum wrote:
On Tue, 2014-09-02 at 13:49 +0200, Petr Vobornik wrote:
On 28.8.2014 20:14, Nathaniel McCallum wrote:
On Tue, 2014-08-19 at 16:46 -0400, Nathaniel McCallum wrote:
Also, remove the attempt to load the objectClasses when absent. This
never makes sense during an add operation.
I still need a review for this. We are trying to get this in 4.0.2.
ACK if comment below doesn't need any change:
Maybe I'm missing something, but why do we do following check: `if
'objectclass' in entry_attrs:`? Shouldn't it be always True? Since
the objectclass is set in LDAPCreate.execute. A pre-callback in an third
party plugin can remove it, but I don't think we should care.
I also thought that was odd, but I cargo-culted it to retain backwards
compatibility. Attached is a version of the patch which doesn't retain
this. I don't care which gets merged.
Freeipa-devel mailing list