patrickpollet / mahara_plugin_auth_cas

3 stars 10 forks source link

Make usernames case-insensitive #3

Closed kd7iwp closed 11 years ago

kd7iwp commented 11 years ago

We had an issue at our institution where this plugin would not authenticate correctly for us. Our LDAP server has upper and lower case letters in the usernames and our CAS server was returning those values. When I modified the CAS server to only return lower-case letters the plugin still would not log our users in. I'm not sure why this is, but when I modified this plugin to use strtolower on both sides of the equation in an 'if' statement inside lib.php (line 132) it solved the problem. Perhaps this affects others negatively, and if so, please do not merge it in. I also added a time stamp to the phpCAS trace debugging as I found that to be somewhat helpful. Thank you for an excellent plugin.

-Jonathan Babbitt