[Date Prev][Date Next] [Chronological] [Thread] [Top]

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

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
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/