[SAMBA4] New objectclass module
simo
idra at samba.org
Fri Jun 2 20:20:35 GMT 2006
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)
- 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
- 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
Aside these minor issues it is a well done module.
Simo.
--
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org
More information about the samba-technical
mailing list