Item12218: case problem after update of LdapContrib

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

Commenting is disabled while not logged in