Bug 4994 - ldap mapping inconsistancy: Net::LDAP returns lowercase names
Summary: ldap mapping inconsistancy: Net::LDAP returns lowercase names
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Authentication (show other bugs)
Version: 3.4
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Galen Charlton
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-07-13 21:23 UTC by Dobrica Pavlinusic
Modified: 2012-10-25 23:09 UTC (History)
3 users (show)

See Also:
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 (1.32 KB, patch)
2010-07-13 21:29 UTC, Dobrica Pavlinusic
Details | Diff | Splinter Review
Signed off patch (1.39 KB, patch)
2011-05-09 22:06 UTC, Chris Cormack
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
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.