Bugs item #2458187, was opened at 2008-12-22 11:44 Message generated for change (Comment added) made by lienhard 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: None 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 ------------------------------------------------------------------------------ _______________________________________________ Dspace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dspace-devel
