Bug 4994

Summary: ldap mapping inconsistancy: Net::LDAP returns lowercase names
Product: Koha Reporter: Dobrica Pavlinusic <dpavlin>
Component: AuthenticationAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: enhancement    
Priority: PATCH-Sent (DO NOT USE) CC: chris, dpavlin, koha.sekjal
Version: 3.4   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: lowercase all mappings before matching them with ldap
Signed off patch

Description Dobrica Pavlinusic 2010-07-13 21:23:24 UTC
This requires us to specify all LDAP mappings in koha-conf.xml in lowercase, instead of original case used withing LDAP.

Compare readability of

<userid      is="hrEduPersonUniqueID"  ></userid>

(which doesn't work) with required (and non-intuitive)

<userid      is="hredupersonuniqueid"  ></userid>
Comment 1 Dobrica Pavlinusic 2010-07-13 21:29:20 UTC Comment hidden (obsolete)
Comment 2 Chris Cormack 2011-05-09 22:06:33 UTC
Created attachment 4070 [details] [review]
Signed off patch
Comment 3 Ian Walls 2011-07-13 19:27:56 UTC
Easy patch to read, simple change being made, still applies.  Marking as Passed QA.
Comment 4 Chris Cormack 2011-07-13 22:44:02 UTC
Pushed, please test
Comment 5 Dobrica Pavlinusic 2011-07-14 15:53:51 UTC
Tested, it works fine, thanks.