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

Re: constraint overlay (ITS#3292)



The first issue that concerned me is the charset constraint
and use of iconv.  The code appears to be incorrect because,
if I understand the iconv API, it doing a conversion between
X and X (where X is the specified charset to constrain to)
instead of between UTF-8 encoded Unicode and X.  Note that
in slapd(8), attribute (and assertion) values of Directory
String syntax (and other character string syntaxes) are
stored as UTF-8 encoded Unicode.

The second issue is that iconv(3) routines are not widely
available (in default installs of supported operating system
platforms).  Hence, I suggest the code only include charset
constraint support of iconv(3) is available.  This implies
a requirement to auto-detect iconv(3) existance to enable
the feature.

The third issue is that the module linking doesn't include
-liconv (which existance should also be auto-detected).

The fourth issue is documentation.  A manpage should be provided.

Another issue I have is whether this should be in slapd/overlays
or in contrib/slapd-modules.  But that's more of issue for
the committers to resolve that you.

Regards, Kurt


At 10:12 PM 8/18/2004, kartik_subbarao@hp.com wrote:
>Full_Name: Kartik Subbarao
>Version: 2.2.15
>OS: Linux
>URL: ftp://ftp.openldap.org/incoming/kartik_subbarao-constraint-040819.tgz
>Submission from: (NULL) (69.240.143.154)
>
>
>I have uploaded Neil Dunbar's constraint overlay, and patches to auxiliary build
>files such as the Makefile.in in the overlays directory, the top level
>configure.in, and portable.h.in. The constraint overlay limits the values which
>can be placed into an attribute, over and above the limits placed by the schema.
>It traps only LDAP adds and modify commands (and only seeks to control the add
>and modify value mods of a modify). The constraint can be specified as a
>character set (e.g. US-ASCII) or as a regular expression.