mahara-contributors team mailing list archive

[Andrew Nicols <684425@xxxxxxxxxxxxxxxxxx>]

Have ensured that all db calls wrap the username in a LOWER() and that
the $username var is strtolowered too.

Given that the usr table has a unique on lower(username), this shouldn't
break anything as mixed-cased usernames aren't supported in any case.

** Changed in: mahara
       Status: New => Fix Committed

** Changed in: mahara
     Assignee: (unassigned) => Andrew Nicols (dobedobedoh)

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/684425

Title:
  remote username doesn't lowercase usernames

Status in Mahara ePortfolio:
  Fix Committed

Bug description:
  One of our many moodles somehow managed to convert their usernames from camelcase to lowercase. They did this after a number of users had SSOd to moodle.
As a result, auth_remote_user has the camelcase variants, and auth/user.php::User->find_by_instanceid_username() doesn't find a match because of the case issues.
As a result, it tries to create a new user, but this fails becuase usr has a unique constraint on lower(username).

I've produced a fix, and will apply in the morning.