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 - This page was cached on 23 Nov 2020 - 13:51.

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