Item12218: case problem after update of LdapContrib

pencil
Priority: Normal
Current State: Closed
Released In: n/a
Target Release: n/a
Applies To: Extension
Component: LdapContrib
Branches: trunk
Reported By: BjoernKautler
Waiting For:
Last Change By: MichaelDaum
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
 
Topic revision: r5 - 19 Nov 2012, MichaelDaum
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy