[SAMBA4] New objectclass module

Andrew Bartlett abartlet at samba.org
Fri Jun 2 21:40:29 GMT 2006


On Fri, 2006-06-02 at 16:20 -0400, simo wrote:
> On Sat, 2006-06-03 at 01:04 +1000, Andrew Bartlett wrote:
> > As per my mail about the objectClass sorting being incorrect in current
> > Samba4, I decided to revive the sorting module.  I did this mostly to
> > try and gain some skill in the new, more complex, async ldb. 
> > 
> > I've tested this with 'make test', which runs a provision and checks the
> > objectclass list on add.  (I need to extend the patched ldap.js I sent
> > yesterday to do a modify).
> > 
> > Any comments?
> 
> Some:
> 
> - if the object has not objectclasses on add then you can probably just
>   return an error, this will be done by the schema module later, but it
>   make no sense to allow to create objects without objectclasses in
>   sam.ldb (in an add operation you can't delete btw)

Yeah, good point.

> - on add you do not need to do the operation, then search, then modify.
>   The objectclass module must clearly be the last one that manipulates 
>   objectclasses, so on add you can just sort the classes and then do the
>   operation as you already have all the classes from the original
>   request

I was doing this to make the codepaths more common, and therefore more
tested.  This is the same reason I did this in the password_hash module.

> - returning "Invalid search filter" in objectclass_search_self() may
>   confuse the user as this is a modify/add operation (you should
>   probably mention an internal error has occurred

OK.

> Aside these minor issues it is a well done module.

Thanks!

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Student Network Administrator, Hawker College  http://hawkerc.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 191 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20060603/e7a1fd46/attachment.bin


More information about the samba-technical mailing list