Item12218: case problem after update of LdapContrib
Priority: Normal
Current State: Closed
Released In: n/a
Target Release: n/a
Applies To: Extension
Component: LdapContrib
Branches: trunk
I updated
LdapContrib on 2012-10-31 from 120112-225 to 120711-279 (debian repository versions).
After updating, the daily foswiki cronjob which calls "sudo -u www-data perl -I bin tools/mailnotify -q" printed out the warning "-
LdapContrib - WARNING: oops, no result looking for user adminuser in LDAP".
"{Ldap}{Exclude}" has the value "WikiGuest,
ProjectContributor,
RegistrationAgent,
UnknownUser,
AdminGroup,
NobodyGroup,
AdminUser, admin, guest".
After I added "adminuser" to "{Ldap}{Exclude}", the warning is gone, so I guess this has something to do with the change "added
CaseSensitiveLogin feature, defaulting to off; performance improvements looking up the ldap cache (by Jayen Ashar)"
Do all "{Ldap}{Exclude}" now have to be specified lowercase?
This would need adaption for every installation.
I guess instead the "{Ldap}{Exclude}" values should also get lowercased if case sensitivity is off.
--
BjoernKautler - 02 Nov 2012
This patch to
lib/Foswiki/Contrib/LdapContrib.pm
fixes it. Will be part of next release.
# create exclude map
- my %excludeMap = map {$_ => 1} split(/\s*,\s*/, $this->{exclude});
+ my %excludeMap = map {$_ => 1, lc($_) => 1} split(/\s*,\s*/, $this->{exclude});
--
MichaelDaum - 11 Nov 2012
Doesn't this need to be dependent on whether case-sensitivity is switched on or off?
--
BjoernKautler - 11 Nov 2012