This is the second bug we've had in the past year that was caused by get_new_username() truncating usernames to 30 characters. (The earlier one was Bug 1486699, which added the param to change the default new username length, and set it to 40 for SAML.)
I wonder if it would make more sense to just update get_new_username() so that it instead truncates usernames to 255 characters, that being the length of usr.username and auth_remote_user.username. -- You received this bug notification because you are a member of Mahara Contributors, which is subscribed to Mahara. Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it! https://bugs.launchpad.net/bugs/1556692 Title: Problem with MNet usernames greater than 30 characters, when $CFG->usersuniquebyusername is used Status in Mahara: In Progress Status in Mahara 1.10 series: Confirmed Status in Mahara 15.04 series: Confirmed Status in Mahara 15.10 series: Confirmed Status in Mahara 16.04 series: In Progress Bug description: This bug occurs when the following set of circumstances are in use: 1. MNet is set up, with users coming *into* Mahara 2. Mnet is set to auto-create user accounts in Mahara 3. $cfg->usersuniquebyusername is set to "True", which means that users are logged in by matching them against usr.username rather than auth_remote_user.remoteusername In this case, users coming from Mnet are provisioned in Mahara with a unique username based on their Moodle username, but truncated to 30 characters. Then on subsequent login attempts, their truncated Mahara username doesn't match their Moodle username, and they do not log in correctly. To manage notifications about this bug go to: https://bugs.launchpad.net/mahara/+bug/1556692/+subscriptions _______________________________________________ Mailing list: https://launchpad.net/~mahara-contributors Post to : [email protected] Unsubscribe : https://launchpad.net/~mahara-contributors More help : https://help.launchpad.net/ListHelp
