Bugs item #2458187, was opened at 2008-12-22 11:44
Message generated for change (Settings changed) made by bollini
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=119984&aid=2458187&group_id=19984

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Authentication
Group: 1.5.1
>Status: Closed
>Resolution: Accepted
Priority: 5
Private: No
Submitted By: Lienhard Jochen (lienhard)
Assigned to: Nobody/Anonymous (nobody)
Summary: netid.toLowerCase() in different classes and methods

Initial Comment:
The netid is usually used for LDAP, but it can also used as persistentid for 
Shibboleth login. The persistenid of Shibboleth is case sensitive, so it is 
better to put the conversion to lover case in the corresponding method, like 
LDAPAuthentication.java and LDAPHierarchicalAuthentication.java, even for the 
findByNetid method, and NOT in EPerson.java

----------------------------------------------------------------------

Comment By: Lienhard Jochen (lienhard)
Date: 2008-12-23 08:53

Message:
Thanks for the fast patch, I think you touched all points.

----------------------------------------------------------------------

Comment By: Stuart Lewis (stuartlewis)
Date: 2008-12-23 08:41

Message:
Hi, Does the attached patch address all the points you raise?. Thanks,
Stuart
File Added:
Fix_for_SF_bug_[2458187]_netid_toLowerCase()_in_different_classes_and_methods.patch

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=119984&aid=2458187&group_id=19984

------------------------------------------------------------------------------
Apps built with the Adobe(R) Flex(R) framework and Flex Builder(TM) are
powering Web 2.0 with engaging, cross-platform capabilities. Quickly and
easily build your RIAs with Flex Builder, the Eclipse(TM)based development
software that enables intelligent coding and step-through debugging.
Download the free 60 day trial. http://p.sf.net/sfu/www-adobe-com
_______________________________________________
Dspace-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dspace-devel

Reply via email to