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
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:
Circulation function:

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
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

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.