[PATCH] New external idmap module

simo idra at samba.org
Tue May 30 22:00:01 GMT 2006


On Tue, 2006-05-30 at 23:59 +0200, Volker Lendecke wrote:
> On Tue, May 30, 2006 at 03:01:53PM -0400, simo wrote:
> > 1) Define whether it is a local or a remote store
> 
> As discussed on IRC, I don't really like this patch. In
> particular the fact that it allows to deactivate the local
> tdb cache. The reason you gave on IRC for this is really
> scary for me, the ability to change ID mappings for existing
> SIDs at will is asking for a whole lot of trouble.
> 
> If this is needed I would much rather see a script forked
> per unknown SID that is then entered into the
> winbindd_idmap.tdb.

No this would not work for people that want a consistent mapping and
want to use their own mapping facility.

And no, requiring to do an rm winbindd_idmap.tdb and restarting samba is
not what I consider a nice and easy way to do it.

Forking a script for each mapping when idmap_external is a local cache
would be overkill.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org



More information about the samba-technical mailing list