[Date Prev][Date Next]
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 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.
At 10:12 PM 8/18/2004, firstname.lastname@example.org wrote:
>Full_Name: Kartik Subbarao
>Submission from: (NULL) (126.96.36.199)
>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.