[SSSD] [PATCH] add store/search/delete interface for custom sysdb objects

Simo Sorce ssorce at redhat.com
Wed Oct 21 00:48:19 UTC 2009


On Wed, 2009-10-21 at 17:15 +0200, Sumit Bose wrote:
> On Tue, Oct 20, 2009 at 07:46:02PM -0400, Simo Sorce wrote:
> > On Mon, 2009-10-19 at 16:42 +0200, Sumit Bose wrote:
> > > Hi,
> > > 
> > > this patch adds a store/search/delete sysdb API for data not related
> > > to
> > > users of groups. The data is stored in cn=custom,cn=domain,cn=sysdb.
> > > The
> > > client must specify a subtree_name and an object_name to save the data
> > > in cn=object_name,cn=subtree_name,cn=custom,cn=domain,cn=sysdb.
> > > 
> > > Please have a look at the sysdb_check_handle_* request, too. I think
> > > it
> > > makes the code more readable and helps to reduce code duplications.
> > 
> > Why are you deleting and re-adding the entry in custom store ?
> > Why not just replace the attributes you want to set ?
> > 
> > Simo.
> > 
> 
> I think it is a typical use case that you want to store a copy of some
> LDAP search result and replacing all exiting data. Maybe a update call
> can be integrated when needed.

Then the name seem misleading, you should s/store/replace/ ?
Or maybe have an option like bool replace true/false ?

Simo.




More information about the sssd-devel mailing list