Re: imporvement to slapo-constraint (ITS#4987)

[Howard Chu <hyc@symas.com>]

manu@netbsd.org wrote:
> Gavin Henry <ghenry@suretecsystems.com> wrote:
>
> > It would be best to move this dicussion to openldap-devel@openldap.org now.
>
> Let's go...
>
> Summarry: I've added a new feature to slapo-constraint for constraining
> a value attribute the the existing values of another attribute. The idea
> is that you can have a catalog of allowed values (eg: for titles: Mr,
> Mrs, Miss) and constraint an attribute to these values
>
> Before working further on this patch (style, dod), I'd like some feeback
> on:
> 1) the way it's implemented: is there rought bugs, or is the logic fine?

I haven't looked at the patch yet.

> 2) configuration syntax: do we keep this one or do we swtich to
> something else?

Something like
	constraint-reference attr1 attr2
might be better.

For simple directives, it's better to use positional parameters and no tag words.
--
  -- Howard Chu
  Chief Architect, Symas Corp.  http://www.symas.com
  Director, Highland Sun        http://highlandsun.com/hyc/
  Chief Architect, OpenLDAP     http://www.openldap.org/project/