Hi, On Tuesday 05 April 2005 12:01, Erik Ableson wrote: > Just came across a minor (but annoying) bug in the documentation. > > The following extract : > > "CONSTRUCTOR > encode => 'none' | 'canonical' | 'base64' > Some values in LDIF cannot be written verbatim and have to > be > encoded in some way: > > 'none' > The default. > > 'canonical' > See "canonical_dn" in Net::LDAP::Util. > > 'base64' > Use base64." > > would lead you to believe that the 'encode' flag applies to all > objects, when in fact it applies only to the DN. > > [....] > > I think that either the encode flag needs to be renamed encodedn > (none|base64|canonical) and the encode flag redirected to apply to > attribute values, but that would break a lot of existing code. > Alternatively, checking the encode flag globally would put the code > back in sync with the documentation.
you are right: the encode flag only applies to the DN. For other attributes the check is not necessary since they can either be written verbatim to the LDIF or get encoded using MIME::Base64 before they are written. I'll change the documentation in CVS to make it more clear. Peter -- Peter Marschall eMail: [EMAIL PROTECTED]
